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

[FEATURE] Modified TopAppBar (1) #81

Merged
merged 5 commits into from
May 13, 2024
Merged

[FEATURE] Modified TopAppBar (1) #81

merged 5 commits into from
May 13, 2024

Conversation

KanuKim97
Copy link
Collaborator

@KanuKim97 KanuKim97 commented May 12, 2024 •
edited
Loading

Description

변경된 디자인 시안에 따른 TopAppBar를 수정했습니다

Content

  1. Home의 TopLevelBar에 디자인 시안과 같은 2개의 action icon이 추가되었습니다.
  2. TopLevelBar의 Title이 디자인 시안에 나온 내용과 동일하게 변경되었습니다.
  3. Icon을 추가했습니다. (IcNotification, IcPeople, IcIosArrowBack)
  4. TopLevelDestination이 아닌 TopBar의 뒤로가기 아이콘 디자인 시안에 나온 아이콘으로 대체되었습니다.
  5. MatchesDetailScreen, TeamSearchScreen에 디자인 시안대로 TopAppBar와 BottomBar를 삽입했습니다.
  6. TeamHistory가 TeamSearch로 네이밍이 변경되었습니다.

To Be Updated Soon...

  1. 디자인 시안과 같이 PlayerDetail에 Transparent Background를 가진 TopAppBar를 삽입할 예정입니다.
  2. MatchesDetail에 Title과 League Logo를 삽입할 예정입니다.

Comment

  • 애플리케이션 구동 후 UI 확인 뒤 Review 부탁드립니다.

KanuKim97 added 2 commits May 12, 2024 14:39
@KanuKim97
[MOD]
1edecb7 
 - mod Top App Bar Icon tintColor
 - mod Top App Bar (match)Text String
 - mod naming TeamHistoryScreen -> Team Search Screen

[ADD]
 - add TopBar Icons
 - add TopBar clickable event
 - add screen bottom bar
@KanuKim97
[MOD]
a1907fc 
 - mod LoginNavigation onBackIconClick
 - mod SignUpScreen, LoginScreen, GnrNavHost onBackIconClick
@KanuKim97 KanuKim97 added the feature New feature or request label May 12, 2024
@KanuKim97 KanuKim97 requested a review from eshc123 May 12, 2024 11:58
@KanuKim97 KanuKim97 self-assigned this May 12, 2024
@eshc123
Copy link
Owner

eshc123 commented May 12, 2024

확인했습니다! BottomBar같은 경우에는 메인에서 세 개의 주 화면(Home, Match, Team)에서만 나오면 될 것 같구요! 상세 보기 등과 같은 뎁스가 하나 더 들어가는 화면에서는 BottomBar가 없어도 될 것 같습니다!
이 부분은 디자인 시안에 다시 반영했는데 기존 디자인 시안 확인을 제대로 하지 못해서 작업을 하신것 같아서 죄송하다는 말씀 전합니다.

eshc123
eshc123 approved these changes May 12, 2024
View reviewed changes
feature/match/src/main/java/com/eshc/goonersapp/feature/match/detail/MatchDetailScreen.kt Outdated
Comment on lines 72 to 89
Scaffold(
topBar = {
TopBar(
title = "",
onBackIconClick = onBackIconClick
)
},
bottomBar = bottomBar
) { paddingValues ->
MatchDetailScreen(
matchDetailUiState = matchData,
onClickChat = onClickChat,
modifier = Modifier
.fillMaxSize()
.padding(paddingValues)
)
}
}
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

MatchDetailScreen의 경우 TopBar가 다른 화면들과 다르게 풀스크린 화면 내에서 다른 UI 컴포넌트 위에 겹쳐서 놓여있는 형태라 고민이 필요할 것 같습니다..!

Copy link
Collaborator Author

@KanuKim97 KanuKim97 May 13, 2024 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

넵 확인했습니다.
해당 파트의 경우 우선 TopBar는 삭제 처리 후에 정해진 시안 따라 처리하겠습니다.
그에 따른 MatchesDetail에 Title과 League Logo를 삽입할 예정입니다. 작업은 보류처리 하겠습니다.

@KanuKim97
[MOD]
24b8f39 
 - mod TeamHistoryScreen, MatchDetailScreen
@KanuKim97
[MOD]
b38c7fa 
 - mod MatchDetailScreen
eshc123
eshc123 approved these changes May 13, 2024
View reviewed changes
Copy link
Owner

@eshc123 eshc123 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

넵, 확인했습니다!

@eshc123 eshc123 merged commit a6c9d9b into dev May 13, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@eshc123 eshc123 eshc123 approved these changes

Assignees

@KanuKim97 KanuKim97

Labels
feature New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@KanuKim97 @eshc123