Wedemy / eggeum-android

나에게 맞는 카페 찾기, 이끔
MIT License
12 stars 0 forks source link

정보수정제안, 카페 메뉴 수정 시에 다이얼로그 Fragment로 변경 #191

Closed kymjaehong closed 10 months ago

kymjaehong commented 10 months ago
height[bot] commented 10 months ago

Link Height tasks by mentioning a task ID in the pull request title or commit messages, or description and comments with the keyword link (e.g. "Link T-123").

💡Tip: You can also use "Close T-X" to automatically close a task when the pull request is merged.

easyhooon commented 10 months ago

지금 확인해봤는데 figma 디자인이랑 만드신 화면이랑 좀 달라서요 확인 해보시구 수정 부탁드려요

figma App
kymjaehong commented 10 months ago

저 명칭 변경이 동일한 string name일 때, 다른 거 잡는 것 같아서 그런 것 같아요. 수정했습니다.

easyhooon commented 10 months ago

사소한 부분들이랑 글자 크기 figma랑 다른 거, dialog 닫는 로직 등등 수정했습니다.

dialog 닫는 부분 바꾼 이유는 아래 링크 참고해주세여 https://stackoverflow.com/questions/61035058/correct-way-to-close-dialogfragment-when-using-navigation-component

kymjaehong commented 10 months ago

피드백 감사합니다 ㅎㅎ