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.

Sign up for GitHub

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

Jump to bottom

[feat] [#31] Map 화면 바텀시트 구현 #54

Merged
merged 16 commits into from
Apr 10, 2024
Merged

[feat] [#31] Map 화면 바텀시트 구현 #54

merged 16 commits into from
Apr 10, 2024

Conversation

easyhooon
Copy link
Collaborator

@easyhooon easyhooon commented Apr 10, 2024 •
edited
Loading

Map 화면 툴팁 바텀시트 기본 바텀시트 검색
바텀시트 검색결과없음 바텀시트 편집
  • MapTopAppBar 의 툴팁 추가 및 Title 클릭시 전체 화면을 차지하는 바텀시트가 나오도록 구현
  • 검색 텍스트에 따른 관심 축제 바텀시트 분기(검색 모드), 검색 결과가 존재할때와 존재하지 않을 때 분기처리
  • 편집 버튼을 클릭하면 편집할 수 있도록(편집 글씨 텍스트와 delete icon 이 너무 작은 것 같다는 생각)
  • UnifestScaffold 추가, Navigation Bar 에 포함되는 화면들의 대한 padding 관련 보일러플레이트 코드 제거
  • compose-keyboard-state 라이브러리를 도입하여, Modifier.removeFocusWhenKeyboardIsHidden() 를 사용하여 TextField 의 입력중 시스템 백버튼을 통해 키보드를 내리는 경우, TextField의 포커스자체를 제거하여 다음에 TextField 를 클릭하면 다시 키보드가 올라올 수 있도록 함.
    -> 다만 해당 Scaffold 내에 포함되는 화면들(Home, Map, Waiting, Menu)만 적용되는 지라, Intro, 관심축제 내에 포함된 TextField 는 적용되지 않음, 근원적인 코드 레벨에서의 해결 필요 (이슈에 등록해두겠습니다)

TODO

  • Map 화면 지도 내에 클러스터링 구현
  • 인기 부스 버튼 선택에 따른 하단 Horizontal Pager Item 분기처리
    -> 하나의 feature 가 너무 헤비해지는 관계로 얘네는 다음 feature 에서 진행하도록 하겠습니다.

easyhooon added 13 commits April 8, 2024 15:12
@easyhooon
[fix] 라이브러리 버전 조정
2252254 
compose-foundation 내에 class 를 찾지 못하는 문제 해결
@easyhooon
[chore] 사용하지 않는 라이브러리 의존성 제거, 라이브러리 의존성 교체, bundle 추가
ba69a8f
@easyhooon
[feat] MapTopAppBar ToolTip 추가
0dbb9cc 
string resource 추가
@easyhooon
[feat] SearchTextField trailing delete icon 추가 및 텍스트필드 비우기 기능 구현
1a2bb87 
ic_delete_gray drawable resource 추가
@easyhooon
[feat] 관심 축제 바텀시트 편집모드 UI 구현
924999a
@easyhooon
[feat] SearchTextField 분리
336a404 
일반 SearchTextField 와 SearchMode 가 존재하는 (뒤로가기, 하단 뷰 안보이게 하는 설정) FestivalSearchTextField 로 분리
@easyhooon
[feat] 학교/축제 검사 결과 리스트 목록 UI 구현
a3903ea 
UnifestOutlinedButton 보완
@easyhooon
[add] compose-keyboard-state 라이브러리 의존성 추가
d8d115a 
system-ui-controller -> compose-system-ui-controller 로 네이밍 변경
@easyhooon
[feat] UnifestScaffold 추가
4dcfcb3 
compose-state-keyboard 라이브러리 적용 및, Scaffold padding 관련 보일러 플레이트 제거
@easyhooon
[feat] FestivalSearchBottomSheet, FestivalSearchResults 구현
f900c45
@easyhooon
[feat] FestivalSearchBottomSheet 항상 최대 높이로 보이도록 설정 조정
dcdbd14 
FlexibleSearchBottomSheet 사용, but issue 존재
@easyhooon
[fix] BottomSheet 하단에 NavigationBar 가 보이는 문제 해결
5da4f2c
@easyhooon
[feat] FestivalSearchTextField SearchMode 분기 처리, FestivalSearchResult…
b822cd7 
…s 연동

code style check success
@easyhooon easyhooon added enhancement New feature or request design tasks releated to design labels Apr 10, 2024
@easyhooon easyhooon requested a review from wjdtkdgns777 April 10, 2024 00:50
@easyhooon easyhooon self-assigned this Apr 10, 2024
@easyhooon easyhooon linked an issue Apr 10, 2024 that may be closed by this pull request
Map 화면 UI #31
Closed
@wjdtkdgns777
Copy link
Collaborator

열심히 읽었는데 바텀시트부터 시작해서 인트로 구조까지 바꾸는 대작업이셨군요.. 수고 너무 많으셨습니다
바텀시트에서 텍스트필드가 커서 위치가 이상하게 잡히는 문제도 있던데 이슈에 함께 올려두겠습니다.

@easyhooon
Copy link
Collaborator Author

easyhooon commented Apr 10, 2024 •
edited
Loading

열심히 읽었는데 바텀시트부터 시작해서 인트로 구조까지 바꾸는 대작업이셨군요.. 수고 너무 많으셨습니다 바텀시트에서 텍스트필드가 커서 위치가 이상하게 잡히는 문제도 있던데 이슈에 함께 올려두겠습니다.

좋은 지적입니다. 저도 고쳐볼려고했는데 딱 가운데로 안가더군여. 이전에 만들었던 앱에서도 아직 못고쳤던 문제인데, 이번 기회 한번 고쳐봐야겠습니다

@easyhooon
Merge branch 'develop' into feature/map-screen-ui
0d6b7b2 
# Conflicts:
#	core/designsystem/src/main/kotlin/com/unifest/android/core/designsystem/component/TopAppBar.kt
#	feature/main/src/main/kotlin/com/unifest/android/feature/main/MainScreen.kt
#	feature/map/src/main/kotlin/com/unifest/android/feature/map/MapScreen.kt
#	feature/map/src/main/kotlin/com/unifest/android/feature/map/navigation/MapNavigation.kt
@easyhooon
[chore] merge 문제되는 부분 해결
68d3223
@easyhooon
[chore] code style check success
5dde027
@easyhooon easyhooon merged commit f613d14 into develop Apr 10, 2024
1 check passed
@easyhooon easyhooon deleted the feature/map-screen-ui branch April 10, 2024 05:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@wjdtkdgns777 wjdtkdgns777 wjdtkdgns777 approved these changes

Assignees

@easyhooon easyhooon

Labels
design tasks releated to design enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Map 화면 UI
2 participants
@easyhooon @wjdtkdgns777