Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

필요없는것으로 보이는 in_game_kkutu.css 제거 #883

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

kmsimproject2
Copy link

없어도 구동이 되는것으로 확인되는 in_game_kkutu.css를 제거한 PR입니다.

@ghost
Copy link

ghost commented May 27, 2021

in_game_kkutu.css가 없어도, in_kkutu.css가 이 파일을 대체하는것 같습니다.

@kmsimproject2
Copy link
Author

in_game_kkutu.css가 없어도, in_kkutu.css가 이 파일을 대체하는것 같습니다.

네, 그점을 확인하여 이 PR을 업로드하였습니다. 자세한 설명 감사드립니다.

Copy link

@kmsimproject kmsimproject left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

comment 잘못썼는데 지우는법을 몰라 이렇게 남깁니다. 죄송합니다.

@ghost
Copy link

ghost commented May 27, 2021

image
만약, html가 아닌, 기본 portal.pug로 사용하신다면, 위에 저 사진 처럼 됩니다.
html를 사용하신다면 in_game_kkutu.css가 상관 없으시겠지만, portal.pug를 사용하신다면,
in_game_kkutu.css를 제거하지 말아주세요!

@kmsimproject
Copy link

image
만약, html가 아닌, 기본 portal.pug로 사용하신다면, 위에 저 사진 처럼 됩니다.
html를 사용하신다면 in_game_kkutu.css가 상관 없으시겠지만, portal.pug를 사용하신다면,
in_game_kkutu.css를 제거하지 말아주세요!

문젯점을 확인하였습니다. portal.pug를 쓰지 않는 분들은 이 PR를 사용하셔도 될것같습니다.

@lshqqytiger
Copy link
Contributor

lshqqytiger commented May 27, 2021

portal.pug 에서 사용하는 파일이므로, 해당 부분을 in_kkutu.css로 수정하거나 in_portal.css에 in_game_kkutu.css의 필요한 부분만 골라내어 추가하여야 합니다.
module.pug를 보면, "in_"+랜더링된 페이지 이름+".css" 파일을 알아서 불러오게 설계되어 있습니다.
따라서, 인게임(kkutu.pug)에서는 in_kkutu.css가 알아서 불러와지지만, 메인화면에서는 in_game_kkutu.css가 없을 경우 in_portal.css만 불러와지면서 일부 내용이 깨지게 됩니다.

@kmsimproject
Copy link

portal.pug 에서 사용하는 파일이므로, 해당 부분을 in_kkutu.css로 수정하는 내용이 추가되어야 합니다.

확인하였습니다. 시간이 되는대로 추가하겠습니다.

@kmsimproject2
Copy link
Author

버그를 수정하였습니다. 확인해보세요.

@kmsimproject2
Copy link
Author

README.md를 잘못 넣었네요 죄송합니다.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants