◐ Shell
reader mode source ↗
Skip to content

scss -> styled component 변환, footer 컴포넌트화, 버그수정 2개#24

Merged
yongchanson merged 8 commits into
react-migrationfrom
yongchanson
Jun 14, 2022
Merged

scss -> styled component 변환, footer 컴포넌트화, 버그수정 2개#24
yongchanson merged 8 commits into
react-migrationfrom
yongchanson

Conversation

@yongchanson

@yongchanson yongchanson commented Jun 11, 2022

Copy link
Copy Markdown
Member

변경사항

  1. 기존 scss -> styled-component로 변환
    • <ThemeProvider>를 추가하여 전역으로 theme를 사용할 수 있도록 함
  2. 다른 문제인데 해설번호를 공유하는 버그수정
    • ex) A문제 2번해설 -> B문제로 이동하면 B문제의 2번해설이 나옴
    • 채린님이 만드신 recoil 그대로 사용함
  3. 이전해설, 다음해설 버튼 - 해당하는 해설이 없으면 비활성화되도록 변경
  4. LoadingPage - 다른 요소가 보이지 않도록 디자인수정

궁금한점

  1. footer의 경우 각 component에 {repoLink}라는 props를 넘겨주는 형태인데, 이것을 전역으로 관리하거나 한번에 보내주는 방법은 없을까요? <ThemeProvider>{theme}처럼 사용하고 싶습니다.
  2. 동적으로 생성되는 div에는 styled-component를 적용하지 못했는데, 다른 방법이 있을까요? 없다면 구조를 변경해야하는데 어떻게 생각하시는지
  3. packjson이 필요하다고 판단되어서 .gitignore에서 삭제하였는데 다른분들은 없는게 맞다고 생각하시나요? -> 회의를 통해 해결
  4. 로딩페이지가 창 크기가 매우 작거나 큰 경우에도 다른요소가 보이지 않았으면 좋겠습니다. -> 길이를 2번 지정하여 해결했습니다. min-width: 2500px; width: 100%;

@yongchanson yongchanson added the 프로젝트 TODO Projects 의 해야할 항목을 완료했을 때 사용합니다. label Jun 11, 2022
@yongchanson yongchanson self-assigned this Jun 11, 2022
@yongchanson yongchanson merged commit 21990f0 into react-migration Jun 14, 2022
@yongchanson yongchanson deleted the yongchanson branch June 14, 2022 06:37

@codeisneverodd codeisneverodd left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Hide comment

고생 많으셨습니다!!
1번은 아래에 footer.js 에 답변 남겼고 나머지에 대해 리뷰 남길게요!
2. 동적으로 생성되는 div 에 styled-componets를 적용하지 못했다는 말을 이해하지 못했어요! 동적으로 생성된다는 것이 어떠한 data를 기반으로 그 데이터를 기반으로 DOM 객체를 새롭게 생성한다는 말일텐데, 생성하는 로직에 styled-component 를 쓰면 된다고 생각해요! 이 부분은 다시 전달 부탁드릴게요
3, 4은 인지 했습니다! 좋은 것 같아요!

4 hidden conversations Load more…
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

프로젝트 TODO Projects 의 해야할 항목을 완료했을 때 사용합니다.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants