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

feat(view): scss 모듈화 도입 #491

Merged
merged 4 commits into from
Sep 30, 2023
Merged

feat(view): scss 모듈화 도입 #491

merged 4 commits into from
Sep 30, 2023

Conversation

KyuTae98
Copy link
Contributor

@KyuTae98 KyuTae98 commented Sep 23, 2023

Related issue

#445

Result

#445 에서 토의했던 SCSS 모듈을 도입했습니다.
타입스크립트에서 module.scss파일을 인식을 못해 src에 global.d.ts를 추가해 파일을 인식할 수 있게했습니다.
image
(모듈화 도입전)

image
(모듈화 도입후)

사진에서 보신것처럼 클래스 네임 뒤에 유니크한 해시값이 붙기 때문에
다른 파일에서 중복된 네이밍이 사용되더라도 문제가 되지 않습니다!

Work list

Discussion

@KyuTae98 KyuTae98 requested review from a team as code owners September 23, 2023 07:46
@KyuTae98 KyuTae98 changed the title Module feat(view): scss 모듈화 도입 Sep 23, 2023
Copy link
Contributor

@2taesung 2taesung left a comment

Choose a reason for hiding this comment

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

e2e test가 있으면 좀 도움이 됐을텐데 ...
styling관련해서 변경 사항은 리뷰에 한계가 있네요 ... ㅠ
(변경 사항이 많고 따져보기 어려워서 ...)
변경 후 내용을 움짤이나 스샷으로 result로 정리해주시면 너무너무너무너무너무 좋을 것 같습니다!

Copy link
Contributor

@ytaek ytaek left a comment

Choose a reason for hiding this comment

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

워. 작업량이 많네요. 고생하셨습니다!

Copy link
Contributor

@yunjin-kim yunjin-kim left a comment

Choose a reason for hiding this comment

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

LGTM 입니다!!

@KyuTae98 KyuTae98 reopened this Sep 30, 2023
@KyuTae98 KyuTae98 merged commit 6db8016 into githru:main Sep 30, 2023
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

4 participants