Conversation
sonms
requested review from
1971123-seongmin,
Hyobeen-Park,
seungjunGong and
vvan2
| val tracker = LocalTracker.current | ||
| var isOpenDialog by remember { mutableStateOf(false) } | ||
|
|
||
| var isFloatingMenuOpen by remember { mutableStateOf(false) } |
There was a problem hiding this comment.
MainFloatingButton 내부에서 관리하던 remember 상태를 제거하고 MainScreen에서 isFloatingMenuOpen을 관리하도록 바꾸면서, FAB의 열림/닫힘 상태의 소유 주체가 명확해진 것 같아요
그래서 MainScreen, MainFloatingMenuOverlay, SoptBottomBar 사이의 책임과 상태 흐름이 더 분명해진 구조로 잘 바꾸신 것 같아요
| Box( | ||
| modifier = Modifier | ||
| .weight(1f) | ||
| ) { |
There was a problem hiding this comment.
NavHost를 Box로 감싼 이유가 뭔지 설명해 주실 수 있으신가여?
There was a problem hiding this comment.
overlay 영역 때문입니다!
기존에는 화면 전체를 어차피 그림자로 덮을거라 상관없었지만
현재 기준으로 바텀바를 제외하고 그림자로 화면을 덮기 + floating 메뉴를 띄워야 때문에 부득이하게 Box로 해당 영역의 경계를 두려고 Box로 묶게 되었습니다!
vvan2
left a comment
vvan2
left a comment
There was a problem hiding this comment.
고생하셨습니다! 사용위치에 따라 모듈 적절하게 이동한것 같습니당
seungjunGong
left a comment
seungjunGong
left a comment
There was a problem hiding this comment.
고생하셨습니다 ! 레이아웃이 좀 까다로운 부분이 있었을거 같은데
잘 수정해주신거 같아요 !!
| ) | ||
| } | ||
|
|
||
| if (navigator.currentTab == Home || navigator.currentTab == MainTab.SoptLog) { |
There was a problem hiding this comment.
p1: 혹시 메뉴 오버레이가 홈과 솝트로그에서만 보여지도록하게한 이유가 있나요?
솝탬프와, 콕찌르기에서도 GNB 에 메뉴 버튼은 보이고 토글 애니메이션은 동작하는데, 메뉴 목록이 안보이니 어색한거 같아요.
There was a problem hiding this comment.
이전 기수에 있던 내용으로 변경을 깜빡했네요! 수정해서 다른 화면에서도 접근 가능하도록 하겠습니다!
…into feat#1560-gnb-add-floating-button
…into feat#1560-gnb-add-floating-button
4 participants
Related issue 🛠
Work Description ✏️
바텀바에 FloatingButton을 위치 시켰습니다
FloatingButton을 통해 메뉴 활성화 시 덮는 그림자 영역을 조절했습니다
이전에는 MainFloatingButton이라는 파일명을 사용하여 menu와 button이 공존했으나 button이 main의 bottombar에 위치하게 되면서 파일 명도 책임을 명확히 하기 위해 이름을 MainFloatingMenuOverlay로 변경했습니다
soptamp와 appjamtamp 모듈에 중복으로 존재하던 NoRippleClickable을 core:common 모듈로 이동했습니다
main 모듈에서도 해당 컴포넌트가 필요해짐에 따라, 공통 UI 유틸리티의 재사용성을 높이고 코드 중복을 제거하기 위해 모듈 간 의존성을 재조정했습니다
Screenshot 📸
To Reviewers 📢
메하하