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.

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

Feature/#52 , #56 뒤로가기 필요한 페이지에 GsTopAppBar 로 뒤로가기 가능한 탑바로 변경 & Navigation Stack 핸들링 하는 코드 삽입 #68

Merged
merged 1 commit into from
May 19, 2024

Conversation

johnhuh619
Copy link
Collaborator

@johnhuh619 johnhuh619 commented May 18, 2024

Issue

Overview (Required)

  • 뒤로가기 가능한 탑바
  • 네비게이션 Stack 핸들링(기능마다 비워서 안드로이드 기본 뒤로가기 사용시 네비게이션 그래프의 모든 페이지 되짚지 않고 깔끔하게 앱 꺼지게)

Links

Screenshot

Before After

@johnhuh619 johnhuh619 added the enhancement New feature or request label May 18, 2024
@johnhuh619 johnhuh619 self-assigned this May 18, 2024
@johnhuh619 johnhuh619 changed the title #52 & #56 뒤로가기 필요한 페이지에 GsTopAppBar 로 뒤로가기 가능한 탑바로 변경 & Navigation Stack 핸… Fefature: #52 & #56 뒤로가기 필요한 페이지에 GsTopAppBar 로 뒤로가기 가능한 탑바로 변경 & Navigation Stack 핸… May 18, 2024
@johnhuh619 johnhuh619 changed the title Fefature: #52 & #56 뒤로가기 필요한 페이지에 GsTopAppBar 로 뒤로가기 가능한 탑바로 변경 & Navigation Stack 핸… Fefature: #52 & #56 뒤로가기 필요한 페이지에 GsTopAppBar 로 뒤로가기 가능한 탑바로 변경 & Navigation Stack 핸들링 하는 코드 삽입 May 18, 2024
@johnhuh619 johnhuh619 changed the title Fefature: #52 & #56 뒤로가기 필요한 페이지에 GsTopAppBar 로 뒤로가기 가능한 탑바로 변경 & Navigation Stack 핸들링 하는 코드 삽입 Fefature#52 & #56 뒤로가기 필요한 페이지에 GsTopAppBar 로 뒤로가기 가능한 탑바로 변경 & Navigation Stack 핸들링 하는 코드 삽입 May 18, 2024
@johnhuh619 johnhuh619 changed the title Fefature#52 & #56 뒤로가기 필요한 페이지에 GsTopAppBar 로 뒤로가기 가능한 탑바로 변경 & Navigation Stack 핸들링 하는 코드 삽입 Fefature/#52 & /#56 뒤로가기 필요한 페이지에 GsTopAppBar 로 뒤로가기 가능한 탑바로 변경 & Navigation Stack 핸들링 하는 코드 삽입 May 18, 2024
@johnhuh619 johnhuh619 changed the title Fefature/#52 & /#56 뒤로가기 필요한 페이지에 GsTopAppBar 로 뒤로가기 가능한 탑바로 변경 & Navigation Stack 핸들링 하는 코드 삽입 Fefature/#52 & #56 뒤로가기 필요한 페이지에 GsTopAppBar 로 뒤로가기 가능한 탑바로 변경 & Navigation Stack 핸들링 하는 코드 삽입 May 18, 2024
@johnhuh619 johnhuh619 changed the title Fefature/#52 & #56 뒤로가기 필요한 페이지에 GsTopAppBar 로 뒤로가기 가능한 탑바로 변경 & Navigation Stack 핸들링 하는 코드 삽입 Feature/#52 & #56 뒤로가기 필요한 페이지에 GsTopAppBar 로 뒤로가기 가능한 탑바로 변경 & Navigation Stack 핸들링 하는 코드 삽입 May 18, 2024
@johnhuh619 johnhuh619 changed the title Feature/#52 & #56 뒤로가기 필요한 페이지에 GsTopAppBar 로 뒤로가기 가능한 탑바로 변경 & Navigation Stack 핸들링 하는 코드 삽입 Feature/#52 , #56 뒤로가기 필요한 페이지에 GsTopAppBar 로 뒤로가기 가능한 탑바로 변경 & Navigation Stack 핸들링 하는 코드 삽입 May 18, 2024
Copy link
Collaborator

@JJongmen JJongmen left a comment

Choose a reason for hiding this comment

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

👍 👍 👍 백 스택까지 관리해주는 거 좋았다

Comment on lines +85 to +89
navController.navigate(Screen.PostsStatus.route) {
popUpTo(navController.graph.startDestinationId) {
inclusive = false
}
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

지금 메인화면이 네비게이션 그래프의 시작점이라서 메인화면 빼고 백스택에서 제거하는 거는 현재는 문제 없거든.

만약에 내가 로그인 화면을 네비게이션 그래프 시작점으로 추후에 두게 된다면 navController.graph.startDestinationId가 로그인 화면 가리킬 수도 있을 것 같아서 이 부분들은 인지하고 내가 나중에 같이 수정해볼게.

@JJongmen JJongmen merged commit e26c40c into main May 19, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

전체 Navigation 로직 변경 글 발행 현황 페이지 기능을 구현한다
2 participants