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

feat: qa 반영 #355

Merged
merged 9 commits into from
Nov 13, 2024
Merged

feat: qa 반영 #355

merged 9 commits into from
Nov 13, 2024

Conversation

siawase17
Copy link
Contributor

🌍 이슈 번호

✅ 작업 내용

  • 경기 리스트, 응원 카운트, 라인업, 응원톡에 있는 로고 배경 제거했습니다.
  • 예정, 종료 뱃지가 분간되도록 습니다.
  • 아무런 응원톡이 작성되지 않은 경우 문구가 띄워지게 했습니다.
    image

♾️ 기타

  • 경기가 없는 경우 메세지를 띄우는 작업은 빈 배열 데이터가 온 후 최종 데이터가 오고 있는 상태라 최종 데이터 분간이 안 돼서 학님이 이 부분은 봐주시기로 했습니다.

@siawase17 siawase17 added the FEAT 새로운 기능 추가 label Nov 9, 2024
@siawase17 siawase17 self-assigned this Nov 9, 2024
Copy link
Member

@ohprettyhak ohprettyhak left a comment

Choose a reason for hiding this comment

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

고생하셨습니다. 코드에 style을 직접적으로 넣어서 사용하고 있지 않아서, 이것을 styles.css.ts 파일에 선언한 후 불러와서 사용하는 방법이 어떨까 싶습니다.

추가로, 현재 Next.js의 Image 컴포넌트에 width, height 값을 넣어주고 있는데 경우에 따라 이미지 비율이 이상해지는 경우가 있더라구요. 이미지에 fill, object-fit: cover 옵션을 넣고, 밖에 div로 감싸 비율을 유지하는 방법을 적용해 보는 것도 좋을 것 같아요.

Copy link
Member

@ohprettyhak ohprettyhak left a comment

Choose a reason for hiding this comment

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

고생하셨습니다 👍

@siawase17 siawase17 merged commit c3337e7 into main Nov 13, 2024
@siawase17 siawase17 deleted the feat/spectator/badge-league-logo-fixes branch November 13, 2024 04:11
hufscheer-saas added a commit to hufscheer-saas/hufscheer-web that referenced this pull request Nov 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
FEAT 새로운 기능 추가
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants