[2주차/Leo] 키워드 제출합니다

[yongyong213]

✅ 워크북 체크리스트

• 모든 핵심 키워드 정리를 마쳤나요?
• 핵심 키워드에 대해 완벽히 이해하셨나요?
• 이론 학습 이후 직접 실습을 해보는 시간을 가졌나요?
• 미션을 수행하셨나요?
• 미션을 기록하셨나요?

✅ 컨벤션 체크리스트

• 디렉토리 구조 컨벤션을 잘 지켰나요?
• pr 제목을 컨벤션에 맞게 작성하였나요?
• pr에 해당되는 이슈를 연결하였나요?(중요)
• 적절한 라벨을 설정하였나요?
• 파트장에게 code review를 요청하기 위해 reviewer를 등록하였나요?
• 닉네임/main 브랜치의 최신 상태를 반영하고 있는지 확인했나요?(매우 중요!)

📌 주안점

[jiyoung02]

LGTM! 수고하셨습니다!! 몇가지 개선사항이 있어서 말씀드리겠습니다.

1. 현재 HomeFragment가 미니 플레이어에 가려져 잘리고있어요 ㅠㅠ
   main_fragmentContainer에 app:layout_constraintBottom_toTopOf="@id/miniPlayer"를 설정하면, 미니 플레이어 위에 배치되어 잘리는 문제가 해결될거에요.

2. 바텀네비게이션의 디자인을 영상과 비슷하게 만들어주세요.
   현재 바텀네비게이션은 선택한 메뉴만 라벨이 나오고, 색상변화가 이루어지지 않고 있습니다. app:labelVisibilityMode="labeled" 속성을 추가해서 라벨이 항상 표시되도록 변경해주세요!!
   또한, 클릭된 아이콘 색상이 바뀌도록 구현해 주시면 추후 프로젝트에서 화면 구현하는데 도움이 될 것입니다!! (저번 스터디 때 보여드린 예시 참고해주세요)

3. AlbumFragment 상단 margin이 너무 넓어요.
   margin이 70이나 들어가있는데, 피그마의 최상단을 기준으로 margin을 준 것으로 추측됩니다. 피그마에서 상단바가 존재한다면, 상단바와의 거리를 기준으로 margin을 맞추는 것이 일반적이에요. 실제 기기 화면과 디자인 가이드가 일치하도록 조정해주세요!!
   (빌드해보고 피그마와 비율을 비교해 보면서 조정하는 것을 추천드립니다)

4. FrameLayout과 ConstraintLayout 중첩 사용을 지양해주세요!
   fragment_album.xml이 ConstraintLayout을 FrameLayout이 감싸는 구조로 이루어져있습니다. 서로 다른 레이아웃을 불필요하게 중첩하면 계층이 깊어져 렌더링 속도가 느려지고, 구조가 복잡해져 수정이 어려워집니다. 하나의 ConstraintLayout으로 통합하는 것이 성능과 관리 측면에서 유리하니 가능하다면 합쳐주세요!!