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.

Sign up for GitHub

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

Jump to bottom

StyledComponent로 옮기기 #115

Closed
seongbin9786 opened this issue Jul 10, 2018 · 3 comments
Closed

StyledComponent로 옮기기 #115

seongbin9786 opened this issue Jul 10, 2018 · 3 comments
Labels
분류: 유지보수 유지보수 우선순위: 최대한 빨리 가장 높은 우선순위
Milestone
더나은기술스택

Comments

@seongbin9786
Copy link
Owner

seongbin9786 commented Jul 10, 2018 •
edited
Loading

JSS가 이슈를 해결하지 않으므로 props를 styling에 사용하는 게 가능한 styled component로 옮겨야 함. 전체 컴포넌트 다 하진 말고 일단 일부만 migration 하는 걸로 시도해보겠음.
@seongbin9786 seongbin9786 added 우선순위: 최대한 빨리 가장 높은 우선순위 분류: 유지보수 유지보수 labels Jul 10, 2018
@seongbin9786 seongbin9786 added this to the 더나은기술스택 milestone Jul 10, 2018
@seongbin9786
Copy link
Owner Author

BoardDetail.js
CommentWriteBox.js
EventHeadlineItem.js
EventItem.js
Header.js
ImagePreview.js
MyPageNav.js
NavDrawer.js
PopupManageItem.js

@seongbin9786
Copy link
Owner Author

seongbin9786 commented Jul 10, 2018 •
edited
Loading

Material-UI에 있는 컴포넌트의 경우 Styled 로 Overriding 하는 방법이 따로 필요하다.

원래 기본 style이 classes로 className을 넘겨서 덮어쓰는 방식이고
styled-component는 className이 없기 때문에,
아래의 방법 중 하나를 사용해야 한다.

  1. && { 어쩌고 저쩌고 } 로 처리하면 우선순위가 높아짐

  2. !important를 항상 뒤에 붙임

@seongbin9786
Copy link
Owner Author

나머지 css는 필요시마다 Styled Component로 이동하는 것으로 함

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
분류: 유지보수 유지보수 우선순위: 최대한 빨리 가장 높은 우선순위
Projects
None yet
Milestone
더나은기술스택
Development

No branches or pull requests
1 participant
@seongbin9786