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] #107-랭킹뷰 수정 (쇼케이스용) #109

Merged
merged 12 commits into from
Dec 3, 2024

Conversation

HyunJaeyeon
Copy link
Member

@HyunJaeyeon HyunJaeyeon commented Nov 30, 2024

👀 연관 이슈

resolved: #107

🛏️ 요약

이번 PR에서 작업한 내용을 간략히 설명해주세요

  • 쇼케이스용 랭킹뷰를 수정하였습니다.

🎁 작업 내용

1. isHost 속성을 기준으로 UI 반영

  • 진행자보다 점수가 높은 사람: isWin -> 기존의 FirstPlace 효과 적용
  • 진행자: isHost -> 기존의 LastPlace 효과 적용
  • 그 외는 else 처리해줌
// 우승자
let isWin = player.score > rankedPlayers.first(where: { $0.isHost })?.score ?? 0 
                    
// RankRow에 isWin, isHost 전달하여 행 UI 적용
                    RankRow(player: player,
                            rank: index + 1,
                            isWin: isWin, 
                            isHost: player.isHost)

2. 순위 명칭 변경

  • <진행자를 이겨라>에선 꼴찌, 그리고 순위로 나타내기보단
    진행자를 이겼는지 혹은 졌는지가 더 중요하다고 판단해서 함 바꿔봤는데 어떠신가요?
  • 아래 스크린샷에서 확인 가능
    private var rankText: String {
        if isWin {
            return "Win"  // 이긴사람
        } else if isHost {
            return "-" // 진행자
        } else {
            return "Lose" // 진사람
        }
    }

📱 작업 스크린샷 (선택)

기존 순위 방식 win-lose 방식
image

🎾 공유사항 (선택)

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요

  • 순위 명칭은 일단 제가 바꿔봤는데 다른 의견 있으시면 편하게 댓글 주세요!
  • 아 그리고 머지는 이전 PR인 Default Player부터 순서대로 해야해요
  • base 브랜치 showcase로 변경 완


격려의 한 마디 (선택)

외마디 비명 (선택)

@HyunJaeyeon HyunJaeyeon added the Feature 새로운 기능 추가 label Nov 30, 2024
@HyunJaeyeon HyunJaeyeon self-assigned this Nov 30, 2024
@HyunJaeyeon HyunJaeyeon linked an issue Nov 30, 2024 that may be closed by this pull request
1 task
@HyunJaeyeon HyunJaeyeon changed the base branch from develop to showcase November 30, 2024 19:49
Copy link
Collaborator

@mosiccan mosiccan left a comment

Choose a reason for hiding this comment

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

새벽에 고많고많~~
이전 pr 문제 해결 후 붙입시다

Comment on lines +33 to +36
RankRow(player: player,
rank: index + 1,
isWin: isWin, // 호스트 보다 위인 사람은 first 효과
isHost: player.isHost)
Copy link
Collaborator

Choose a reason for hiding this comment

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

승리조건 로직 확인!

Comment on lines +174 to +176
} else if isHost {
return "-"
} else {
Copy link
Collaborator

Choose a reason for hiding this comment

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

진행자의 경우 '-' 한글자만 있어서 왼쪽으로 좀 치우쳐지는데
디자이너한테 한번 물어봅죠

Copy link
Member Author

Choose a reason for hiding this comment

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

좋습니다!

@HyunJaeyeon HyunJaeyeon merged commit 1816c58 into showcase Dec 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature 새로운 기능 추가
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feat] RankingView 수정 (ShowCase)
2 participants