-
Notifications
You must be signed in to change notification settings - Fork 0
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
chore: 공용 스타일 파일 디테일 수정 #19
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🐧님 언더바 때문에 이틀 가량 고생하신 것 같아요...
😂😂 감사합니다. 고생 많으셨어요!
나머지 파일은 제가 언더바 제거해서 올리겠습니다.
@import url('https://cdn.jsdelivr.net/gh/orioncactus/[email protected]/dist/web/variable/pretendardvariable.min.css'); | ||
@import '@/_asset/reset.scss'; | ||
@import '@/asset/reset.scss'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
✋질문 있습니다.
기존 방식이 globals에서 reset, variables를 import해서 관리하는 방식이었잖아요?
variables는 config에서 전역으로 사용 가능하게 바꾸신 거고요.
reset도 config에 추가해서 사용하는 게 가능할까요??
어차피 variables처럼 전역으로 사용되는 파일이니까??
$radi-category-small: 6px; | ||
$radi-category-filter: 100px; | ||
$radi-modal: 16px; | ||
$radius-common: 8px; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
섬세함 👍
📝 Description
asset 명 수정 및 variables 파일 수정
📸 Screenshot
📢 Notes