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

각종 링크 수정 및 scrollToTop 버튼 구현(3/15 프론트엔드 피드백 반영) #14

Merged
merged 9 commits into from
Mar 18, 2024

Conversation

nestiank
Copy link
Collaborator

@nestiank nestiank commented Mar 17, 2024

  • 브로슈어, 판례, 기사 링크 교체
  • WhiteButtonTitle element의 링크 삭제
  • 서명운동 참여하기 버튼을 누르면 보기 좋게 하단으로 이동하도록 링크 및 CSS 수정
  • 검색 버튼 색상 변경
  • scrollToTop 버튼 구현 - feat: 스크롤 버튼 만들기 #9
  • 기타
    • 컴포넌트명 오류 수정

- 브로슈어, 판례, 기사 링크 교체
- WhiteButtonTitle element의 링크 삭제
- 서명운동 참여하기 버튼을 누르면 보기 좋게 하단으로 이동하도록 링크 및 CSS 수정
@nestiank nestiank added the enhancement New feature or request label Mar 17, 2024
@nestiank nestiank requested a review from koremp March 17, 2024 05:47
@nestiank nestiank self-assigned this Mar 17, 2024
Copy link

vercel bot commented Mar 17, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
call22nd ✅ Ready (Inspect) Visit Preview 💬 Add feedback Mar 18, 2024 7:54am

@nestiank
Copy link
Collaborator Author

nestiank commented Mar 17, 2024

데스크톱에서 scrollToTop 버튼이 너무 오른쪽으로 붙는데, 어떻게 하면 좋을까요?

@koremp
Copy link
Collaborator

koremp commented Mar 17, 2024

데스크톱에서 scrollToTop 버튼이 너무 오른쪽으로 붙는데, 어떻게 하면 좋을까요?

max width 360px짜리 div 태그 추가하면 되지 않을까요?

@koremp
Copy link
Collaborator

koremp commented Mar 17, 2024

데스크톱에서 scrollToTop 버튼이 너무 오른쪽으로 붙는데, 어떻게 하면 좋을까요?

max width 360px짜리 div 태그 추가하면 되지 않을까요?

window 객체를 사용해서 하면 window width resize 될 때마다 가변값으로 fixed position 값을 변경해주면 될 것 같네요. useEffect 훅을 사용해서 컴포넌트가 마운트 될 때 window의 width 값을 가져와서 초기 값 설정해주면 될 것 같네요. 지금 하려다가 막혀서... 내일 또 해보겠습니다

@koremp
Copy link
Collaborator

koremp commented Mar 18, 2024

데스크톱에서 scrollToTop 버튼이 너무 오른쪽으로 붙는데, 어떻게 하면 좋을까요?

max width 360px짜리 div 태그 추가하면 되지 않을까요?

window 객체를 사용해서 하면 window width resize 될 때마다 가변값으로 fixed position 값을 변경해주면 될 것 같네요. useEffect 훅을 사용해서 컴포넌트가 마운트 될 때 window의 width 값을 가져와서 초기 값 설정해주면 될 것 같네요. 지금 하려다가 막혀서... 내일 또 해보겠습니다

Next.js의 dynamic import를 사용해서 Reference Error: window is undefined 에러 해결했는데 css 중에서 right만 안잡히네요. 이것만 해결하면 될 것 같네요

@koremp
Copy link
Collaborator

koremp commented Mar 18, 2024

window의 width가 360px 이상일 때 화면이 너무 가독성이 좁아지는 것 같아서 우선 mainWrapper로 width 360px를 설정해봤습니다.

image

@koremp
Copy link
Collaborator

koremp commented Mar 18, 2024

window width가 변경될 때마다 right 25px 고정되게 수정했습니다.
화면 기록 2024-03-18 오전 10 13 00

@nestiank
Copy link
Collaborator Author

scrollToTop 버튼에 border 대신 shadow 추가를 하였습니다.
코드에 문제가 없는 것 같습니다 :)

@nestiank nestiank merged commit 208f9a9 into develop Mar 18, 2024
4 checks passed
@koremp koremp deleted the feat/link-fix branch March 21, 2024 12:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants