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

[#97] Style: 메인페이지 ui 수정 #98

Conversation

ienrum
Copy link
Collaborator

@ienrum ienrum commented Jun 14, 2024

💬 Issue Number

closes #97

🤷‍♂️ Description

작업 내용에 대한 설명

  • svg 추가
  • 게시글 리스트의 아이템 html 구조 변경
  • popup, floating button, gnb 컴포넌트 를 sticky or fixed 로 변경

📷 Screenshots

작업 결과물

-.Chrome.2024-06-14.16-06-55.mp4

👻 Good Function

팀원에게 공유하고 싶은 함수나 코드 일부

📋 Check List

PR 전 체크해주세요.

  • Merge 하는 브랜치가 올바른가?
  • 코딩컨벤션을 준수하는가?
  • PR과 관련없는 변경사항이 없는가?

📒 Remarks

팀원이 코드리뷰 시 주의할 점 또는 말하고 싶은 점 특이사항

@ienrum ienrum added the 🎨 Style style수정 label Jun 14, 2024
@ienrum ienrum self-assigned this Jun 14, 2024
Copy link
Collaborator

@hyeonjinan096 hyeonjinan096 left a comment

Choose a reason for hiding this comment

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

수고하셨습니당
image
모바일 버전이 아닌 웹 화면에 경우 이렇게 floating button이 밖으로 나가서 fixed를 안했던 기억이 있는데 모바일 서비스만 제공하니 상관없을까요?

@ienrum
Copy link
Collaborator Author

ienrum commented Jun 18, 2024

수고하셨습니당 image 모바일 버전이 아닌 웹 화면에 경우 이렇게 floating button이 밖으로 나가서 fixed를 안했던 기억이 있는데 모바일 서비스만 제공하니 상관없을까요?

아하 그러네요, 확인 감사합니다.
floating 버튼 ui 가 gnb 바에 포함되는 기획으로 변경되어서, 관련해서 고쳐보겠습니다.

@HoberMin HoberMin merged commit 9658c04 into #90/feat/fetch-to-tanstack-mainpage/chae Jun 18, 2024
1 check failed
@HoberMin HoberMin deleted the #97/style/restyle-mainpage/chae branch June 18, 2024 14:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🎨 Style style수정
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants