-
Notifications
You must be signed in to change notification settings - Fork 1
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
fix: 무한 설정 페이지 버그 수정합니다. #265
Conversation
…to feat/profile
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
@@ -48,7 +48,7 @@ export default function Page() { | |||
<div> | |||
<HeaderBar> | |||
<HeaderBar.LeftContent> | |||
<BackButton onClickBack={() => router.back()} /> | |||
<BackButton onClickBack={() => router.push('/')} /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
기록 시각화 기준 거리 수정 후, 설정 페이지로 이동할 때 이벤트를 replace로 바꿔주시는건 어떨까요?
- 홈 > 설정 > push > 기록 시각화 기준 거리 > replace > 설정
- 결국은 홈 > 설정 > 설정의 히스토리만 남게 됩니다. (기록 시각화 기준 거리는 대체됩니다)
기록 상세에서도 동일한 흐름을 가져가고 있는데요, 요 방법으로 우선 적용해놓고 페이지 중복 문제는 한번에 해결해보시죠!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
혹은 기준 거리 수정 후 > 뒤로가기 해도 괜찮겠네요!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Jungjjeong 좋은 의견 감사합니닷!!! 🙇🏻
🤔 어떤 문제가 발생했나요?
🎉 어떻게 해결했나요?
📷 이미지 첨부 (Option)