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.

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

[SP0] 메인 페이지 수정사항 반영 #379

Merged
merged 31 commits into from
Mar 24, 2024
Merged

Conversation

eonseok-jeon
Copy link
Member

@eonseok-jeon eonseok-jeon commented Mar 24, 2024

Summary

  • 아 그 일단 급한 거 먼저 쓰자면
    activity, part, team, review 탭 smooth scroll 관련해서
    왜인지 scroll-behavior: smooth; 가 안 먹히는 거예요
    그래서 스크롤 관련이라 다른 거랑 충돌나지 않을테니 괜찮겠지 하고 일단 !important 박아뒀는데,
    어떻게 생각하시나요,,?
    근데 제가 방법을 찾아보려고 열심히 구글링 했는데
    도저히 작동 안 하는 이유를 찾을 수 없었습니다ㅜ

  • 워딩들 수정했고, 반응형 구현했습니다!

  • file changed 보시면 아시겠지만 사실 거의 CSS 수정이 대부분이고 그나마 구조 살짝 변경된 부분이 있다면src/views/MainPage/components/BottomLayout/index.tsx
    여깁니다.
    최신 뉴스랑 모집 공고를 분리시켜야 해서 살짝 봐꿔줬습니다.

  • 이제 해야할 게

  • team section에서 makers 이미지 변경.
  • team section 모바일뷰 디자인 수정.
  • 모집 공고 버튼 다시 되돌리기.
  • 워딩 수정.
  • 모바일 뷰 소제목 추가.
  • value section 각 항목마다 딱딱 멈추게 하기. (다음 스프린트로 넘깁시다,,!)
  • 반응형 구현하다 스트레스 받아서 사망할 거 같아요
    논의가 필요한 게 break point를 현재 1440 786 376 이렇게 했지만
    공홈 디자인이 이거에 딱 맞지가 않습니다.
    각 섹션 마다 디자인 깨지는 포인트가 달라서 따로 처리해야 되는 부분들이 많았습니다.
    현재 구현된 코드를 보면 break point를 428px로 설정한 부분도 많고 520, 976 등등 너무 많아요.
    그래서 폰트는 376에서 바뀌는데 이미지 크기는 428에서 바뀌는 등
    연결이 부자연스러운 부분이 꽤나 많은 거 같습니다.
    최대한 많이 잡아보려고 노력했지만 쉽지 않았습니다.
    아예 디자인을 376에서 다 바뀌도록 수정하거나
    각각의 상황에 맞게 수정하거나 등이 필요할 거 같습니다.
    근데 이거 하려면 대공사 될 거 같아서 선뜻 하자고 하기가 어렵네요.
    별로 하고 싶지도 않,,
    그 딱히 별 특별한 문제가 없다면 일단은 이대로 가도 좋을 거 같아요.

  • 추가로 하고 싶은 건

스크린샷 2024-03-24 오후 7 11 55 스크린샷 2024-03-24 오후 7 12 04 스크린샷 2024-03-24 오후 7 12 13

공통된 느낌이 있는데 이에 대해서 재사용 가능한 컴포넌트를 분리해보고 싶네요.

@eonseok-jeon eonseok-jeon requested a review from lydiacho March 24, 2024 01:52
@eonseok-jeon eonseok-jeon self-assigned this Mar 24, 2024
@eonseok-jeon eonseok-jeon linked an issue Mar 24, 2024 that may be closed by this pull request
10 tasks
Copy link

height bot commented Mar 24, 2024

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.

Copy link
Member

@lydiacho lydiacho left a comment

Choose a reason for hiding this comment

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

아 진짜 반응형 지옥 너무너무너무너무너무너무너무너무 수고많으셨습니다
피눈물이 묻어있는 PR이네요
smooth scroll 관련한 해결책 코멘트에 적어놓았으니 참고 부탁드리구,

메인페이지 반응형 말씀하신 부분 아주 공감합니다. 제가 반응형 브레이크포인트를 통일하면 좋겠다. 브레이크 포인트가 통일이 안되어있어서 중간에 튀는 부분들이 있다 라고 제시했던 원인이 바로 이 문제였어요 🥲🥲

특히 메인페이지는 레이아웃이 복잡하고 포함되어있는 콘텐츠가 많아서 단순히 기기별 대응 브레이크 포인트 뿐만 아니라 훨씬 세세하게 쪼개지게 되는데요
드리고 싶은 말씀은 일단 급한 건 아니면 지금 혼자서 다 정리하지 않으셨음 해요 ㅠ
얼마나 막연한 일인지 알기에...
메인페이지 브레이크 포인트 관련해서는 다음 회의때 제안해서, 다음 스프린트때 디자이너 분들께 먼저 브레이크 포인트 통일 요청드리고, 이후에 저희가 받아서 싹 정리해주면 될 것 같아요

+) value section 스크롤 포인트 멈추게 하는거 무조건 다음스프린트로 넘기자!!!!! 동의!!!!!!!!

src/lib/styles/global.ts Outdated Show resolved Hide resolved
src/views/MainPage/components/BottomLayout/index.tsx Outdated Show resolved Hide resolved
src/views/MainPage/components/OwnOrganization/index.tsx Outdated Show resolved Hide resolved
@eonseok-jeon eonseok-jeon merged commit 1785519 into develop Mar 24, 2024
1 check passed
@eonseok-jeon eonseok-jeon deleted the fix/#377_modify branch March 24, 2024 15:50
@lydiacho lydiacho added this to the 4기 milestone Mar 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[SP0] 메인 페이지 수정사항 반영
2 participants