-
Notifications
You must be signed in to change notification settings - Fork 17
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
[FE] 유저 테마 컬러 변경 기능 추가 #1399
[FE] 유저 테마 컬러 변경 기능 추가 #1399
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.
고생 많으셨습니다 실제 작동하는 모습을 얼른 보고 싶네요!
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.
파스텔톤으로 색깔 잘 뽑으신 것 같습니다. 고생하셨습니다~
setShowColorPicker: React.Dispatch<React.SetStateAction<boolean>>; | ||
}> = ({ showColorPicker, setShowColorPicker }) => { | ||
const savedColor = localStorage.getItem("mainColor"); | ||
const defaultColor = "#9747ff"; |
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.
여기에 defaultColor "#9747ff" 같은 고유 컬러 값들은 contants 폴더에 따로 관리하는 것도 괜찮을 것 같네요.
생각보다 여러 파일에서 사용 중으로 보여서 분리해서 차후를 위해 관리하는게 도움될 것 같습니다.
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.
테마 컬러 작업하느라 고생 많으셨습니다. 실제로 동작해서 봤는데, 잘 동작하는 것 같아서 좋았습니다.
svgr rollup이라는 라이브러리도 처음 들어봐서, 오랜만에 PR이 저에게 도움이 많이 되었습니다.
[아이디어]
색상 밝기에 따라서 textColor도 동적으로 바뀌면 더 이상적일 것 같아서 글로 아이디어 적어봅니다.
아마 textColor는 따로 변수로 처리 안된 부분이 많을 거라 이 부분은 작업하는데 노가다일 수도 있겠네요.
언젠가 마음이 동하실때 해보는 것을 추천드릴게요. (하면 좋을거 같다는거지 하라는거 아님..!)
지선생님 의견을 구해서 도움이 될만한 코드도 같이 첨부합니다
해당 사항 (중복 선택)
설명
#1397
사용자가 테마 컬러를 변경할 수 있는 기능을 추가하였습니다.