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

[Others] 유저 토너먼트 전적 페이지 UI 변경 #1160 #1164

Conversation

Clearsu
Copy link
Collaborator

@Clearsu Clearsu commented Dec 14, 2023

📌 개요

  • 기존 디자인을 조금 더 화려하게 수정하였습니다.
  • 기존
image
  • 변경
image

Dec-14-2023 17-56-29
(임시로 mock 데이터 사용하여 녹화하였습니다)

💻 작업사항

  • 다른 페이지와 통일(모두 영문)을 위해 페이지 타이틀 명 명예의 전당 -> Hall of Glory 로 변경
  • 가장 윗 레이어의 우승자 box-shadow 추가
  • 리그 별 테두리 색상 적용
  • 타이틀 glow 애니메이션 추가

@Clearsu Clearsu added the user 일반 사용자가 사용하는 서비스에서 발생한 issue label Dec 14, 2023
@Clearsu Clearsu added this to the 42gg-5th-fe milestone Dec 14, 2023
@Clearsu Clearsu self-assigned this Dec 14, 2023
@Clearsu Clearsu linked an issue Dec 14, 2023 that may be closed by this pull request
Copy link
Collaborator

@joonho0410 joonho0410 left a comment

Choose a reason for hiding this comment

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

좋습니다~!

indexDiff > -2 && indexDiff < 2 && applyStyle()
}`}
indexDiff > -2 && indexDiff < 2 && applyLayerStyle()
} ${applyTypeStyle(type)}`}
Copy link
Collaborator

Choose a reason for hiding this comment

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

CSS 파일에서 대문자로 &.ROOKIE와 같이 만들어서 styles[type] 이런방식으로 바로 접근할 수는 없을까요 ??

Copy link
Collaborator Author

@Clearsu Clearsu Dec 14, 2023

Choose a reason for hiding this comment

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

오 좋은 방법이네요ㅎㅎ 수정하도록 하겠습니다

@greatSweetMango greatSweetMango merged commit 989ea9b into main Dec 15, 2023
1 check passed
@Clearsu Clearsu deleted the others/유저-토너먼트-전적-페이지-ui-변경-#1160 branch December 15, 2023 05:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
user 일반 사용자가 사용하는 서비스에서 발생한 issue
Projects
Status: in-review
Development

Successfully merging this pull request may close these issues.

[Others] 유저 토너먼트 전적 페이지 UI 변경
3 participants