Closed eonseok-jeon closed 7 months ago
Link Height tasks by mentioning a task ID in the pull request title or commit messages, or description and comments with the keyword
link
(e.g. "Link T-123").💡Tip: You can also use "Close T-X" to automatically close a task when the pull request is merged.
Summary
아 그 일단 급한 거 먼저 쓰자면 activity, part, team, review 탭 smooth scroll 관련해서 왜인지 scroll-behavior: smooth; 가 안 먹히는 거예요 그래서 스크롤 관련이라 다른 거랑 충돌나지 않을테니 괜찮겠지 하고 일단 !important 박아뒀는데, 어떻게 생각하시나요,,? 근데 제가 방법을 찾아보려고 열심히 구글링 했는데 도저히 작동 안 하는 이유를 찾을 수 없었습니다ㅜ
워딩들 수정했고, 반응형 구현했습니다!
file changed 보시면 아시겠지만 사실 거의 CSS 수정이 대부분이고 그나마 구조 살짝 변경된 부분이 있다면
src/views/MainPage/components/BottomLayout/index.tsx
여깁니다. 최신 뉴스랑 모집 공고를 분리시켜야 해서 살짝 봐꿔줬습니다.이제 해야할 게
[x] team section에서 makers 이미지 변경.
[x] team section 모바일뷰 디자인 수정.
[x] 모집 공고 버튼 다시 되돌리기.
[x] 워딩 수정.
[x] 모바일 뷰 소제목 추가.
[ ] value section 각 항목마다 딱딱 멈추게 하기. (다음 스프린트로 넘깁시다,,!)
반응형 구현하다 스트레스 받아서 사망할 거 같아요 논의가 필요한 게 break point를 현재 1440 786 376 이렇게 했지만 공홈 디자인이 이거에 딱 맞지가 않습니다. 각 섹션 마다 디자인 깨지는 포인트가 달라서 따로 처리해야 되는 부분들이 많았습니다. 현재 구현된 코드를 보면 break point를 428px로 설정한 부분도 많고 520, 976 등등 너무 많아요. 그래서 폰트는 376에서 바뀌는데 이미지 크기는 428에서 바뀌는 등 연결이 부자연스러운 부분이 꽤나 많은 거 같습니다. 최대한 많이 잡아보려고 노력했지만 쉽지 않았습니다. 아예 디자인을 376에서 다 바뀌도록 수정하거나 각각의 상황에 맞게 수정하거나 등이 필요할 거 같습니다. 근데 이거 하려면 대공사 될 거 같아서 선뜻 하자고 하기가 어렵네요. 별로 하고 싶지도 않,, 그 딱히 별 특별한 문제가 없다면 일단은 이대로 가도 좋을 거 같아요.
추가로 하고 싶은 건
공통된 느낌이 있는데 이에 대해서 재사용 가능한 컴포넌트를 분리해보고 싶네요.