[Feat] 유저 단축어 등급 기능 추가 #412
Conversation
- 뱃지 이미지 추가 - 프로필 이미지 뱃지 이미지로 대체
- super image 추가 - grade alerter 추가
There was a problem hiding this comment.
데이터 구조나 함수 되게 깔끔하게 구현해 주신 것 같아요! 수고하셨습니다 👍🏻
아래 피드백만 검토 부탁드립니다.
- alert을 제외한 다른 아이콘 해상도가 너무 낮게 설정되어 있는 것 같아요. 작은 아이콘인데도 낮은 해상도로 보여서 해상도를 높인 에셋으로 재등록해주셨으면 좋겠습니다!
- Figma에서 export하실 때 1x말고 3x설정으로 뽑아주시면 해결될 것 같습니당
- 라이트모드일 때는 괜찮은데, 다크모드에서 실행했을 경우 Alert창의 텍스트 및 xmark가 흰색인 Alert배경과 너무 대비가 약해 사용자가 불편해 할 것으로 예상됩니다. 두 가지 해결방향이 생각나는데 어떤 방향이 더 좋을까요? 개인적으로는 다크모드에서 갑자기 큰 흰색 창이 나타나면 불편하게 느낄 수 있을 것 같아 두 번째 해결방법이 좋을 것 같습니다
- Alert배경은 다크모드에서도 흰색으로 유지하고 텍스트 및 xmark 색상을 진하게 변경하자.
- 텍스트 및 xmark 색상은 현행대로 gray5로 유지하고 Alert배경색을 background회색으로 설정하자. -> 배경색과 같지만 배경이 Dimmed 되어있어서 두 색상 간의 차이를 인식할 수 있을 거라고 생각합니다! 잘 인지가 안 된다면 더 밝은 회색으로 설정하는 방향도 좋습니다.
| Dark | Light |
|---|---|
 |
 |
|
혹시 지민님이 원하시는 뷰 전환 효과가 이게 맞을까요? 2023-03-01.4.18.45.mov |
와우... 엄청난 코드리뷰네요 👍🏻 |
꼼꼼한 리뷰 감사합니다!
2번 괜찮다고 생각되시면 바로 수정 푸시하겠습니다! |
Co-authored-by: Soi (Jiwon Lee) <zest1923@gmail.com>
Co-authored-by: Soi (Jiwon Lee) <zest1923@gmail.com>
Co-authored-by: Soi (Jiwon Lee) <zest1923@gmail.com>
|
@JMM00 변경된 색상 구분 잘 되어서 괜찮을 것 같습니다! |
Co-authored-by: Soi (Jiwon Lee) <zest1923@gmail.com>
네!!!!! 맞아요!!!!! 어디서 해당 애니메이션을 넣어야하죠,,,.????ㅠㅠㅠㅠㅠ |
꼼꼼한 리뷰 감사합니다~!!!!!!! 제가 놓친 부분들까지 꼼꼼하게 잡아주셨네요ㅠㅠ 제가 마음만 급해서..부족한 pr을 올린 것 같아 죄송합니다. 앞으로는 좀 더 꼼꼼하게 확인 후 진행하겠습니다!! 감사합니다 |
- 뱃지 이미지 추가 - 프로필 이미지 뱃지 이미지로 대체 resolve conflict
resolve conflict
- super image 추가 - grade alerter 추가 resolve conflict
resolve conflict
Co-authored-by: Soi (Jiwon Lee) <zest1923@gmail.com>
Co-authored-by: Soi (Jiwon Lee) <zest1923@gmail.com>
Co-authored-by: Soi (Jiwon Lee) <zest1923@gmail.com>
Co-authored-by: Soi (Jiwon Lee) <zest1923@gmail.com>
…tGradeSystem' Conflicts: HappyAnding/HappyAnding/TextLiteral.swift HappyAnding/HappyAnding/Views/Components/MyShortcutCardListView.swift HappyAnding/HappyAnding/Views/ExploreCurationViews/ReadUserCurationView.swift HappyAnding/HappyAnding/Views/ShortcutDetailViews/ReadShortcutView/ReadShortcutHeaderView.swift
|
푸시 이후에는 리베이스 말고 머지로 최신 디벨롭 반영하는게 좋다고 하네요 |
Co-authored-by: Soi (Jiwon Lee) <zest1923@gmail.com>
관련 이슈
구현/변경 사항
스크린샷
iPhone SE
Simulator.Screen.Recording.-.iPhone.SE.3rd.generation.-.2023-02-28.at.23.44.45.mp4
iPhone14 pro
Simulator.Screen.Recording.-.iPhone.14.Pro.-.2023-02-28.at.23.43.10.mp4
To Reviewer