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

[#191] Fix: 스타일 및 버그 수정 #192

Merged
merged 8 commits into from
Apr 23, 2024

Conversation

Heeeera
Copy link
Contributor

@Heeeera Heeeera commented Apr 6, 2024

💬 Issue Number

closes #191

🤷‍♂️ Description

  • 종료된 게임 페이지 리뷰 작성 버튼 노출 조건이 잘못 되어있던 것 수정
  • Modal, Select z-index 속성 수정
  • 보드게임 상세, 찜한 게임 목록, 종료된 게임 목록 페이지 이미지 object-fit 속성 수정
  • 모임 상세 페이지 참여하기 버튼 비활성화 조건 수정
  • 모임 상세 페이지 모임 설명 white-space 속성 수정
  • 모임 목록 페이지 Suspense 추가
  • 알림 목록 아이템 text-align 속성 수정
  • 리뷰 작성 완료 후 이동하는 페이지 url 수정

📷 Screenshots

모달

보드게임 상세 이미지

모임 설명

📋 Check List

PR 전 체크해주세요.

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

@Heeeera Heeeera added the 🐞 Fix 버그 등의 오류 수정 label Apr 6, 2024
@Heeeera Heeeera self-assigned this Apr 6, 2024
@Heeeera Heeeera changed the title [#191] 스타일 및 버그 수정 [#191] Fix: 스타일 및 버그 수정 Apr 6, 2024
Copy link
Collaborator

@HoberMin HoberMin left a comment

Choose a reason for hiding this comment

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

희라님 군데군데 작업하느라 고생많이 하셨습니다 !
변경사항들에 대해서 간단한 리뷰들만 작성해서 반영되면 resolve직접 해주셔도 좋을 것 같아요

그리고 추가적으로 기억이 잘 안나서 그러는데 맥os에서 select어떻게 처리하기로 했었죠..? 맥에서는 기본 select로 동작해서 스타일이 안입혀지네요

src/components/EndGameListItem/index.tsx Show resolved Hide resolved
src/components/EndGameListItem/index.tsx Outdated Show resolved Hide resolved
@Heeeera
Copy link
Contributor Author

Heeeera commented Apr 6, 2024

@HoberMin

그리고 추가적으로 기억이 잘 안나서 그러는데 맥os에서 select어떻게 처리하기로 했었죠..? 맥에서는 기본 select로 동작해서 스타일이 안입혀지네요

사진은 iOS 사파리인데 맥 사파리에서도 사진처럼 나온다면 문제 없을 것 같아요!

Copy link

cloudflare-workers-and-pages bot commented Apr 15, 2024

Deploying boardsignal with  Cloudflare Pages  Cloudflare Pages

Latest commit: f292542
Status: ✅  Deploy successful!
Preview URL: https://e2301153.team-civilwar-boardsignal-fe.pages.dev
Branch Preview URL: https://191-fix-page-style-and-bug-f.team-civilwar-boardsignal-fe.pages.dev

View logs

@Heeeera Heeeera merged commit 088559b into main Apr 23, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐞 Fix 버그 등의 오류 수정
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Fix] 모달, 모임 상세 설명, 리뷰 작성 버튼, 모임 목록 등 스타일 및 버그 수정
2 participants