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

[FE] UT피드백에 의한 편집 페이지 새로고침 로직 문제를 해결한다 #886

Merged
merged 6 commits into from
Oct 24, 2024

Conversation

skiende74
Copy link
Contributor

@skiende74 skiende74 commented Oct 24, 2024

❗ Issue

✨ 구현한 기능

  • (로직) 편집페이지 새로고침시 가까운역 안뜨던문제해결.
    -> 방법 = 이 store도 로컬스토리지관리

  • (로직) 디폴트방이름 1번부터시작하게.
    -> 방법 = 예시용체크리스트를 카운팅안하게해서 1번부터시작하게

  • (접근성) 방기본정보 인풋태그 포커스(탭)안되던 현상 해결
    -> 원인 = 원래 되는게 맞는거라 원인확인해본결과 포커스트랩이 원인이였습니다.
    모달이 켜져야 포커스트랩이 발동하도록되어있어야하는데 모달꺼져있을때도 포커스트랩이 켜져있어서 모달 오픈시만 포커스되도록 고쳐서해결했습니다.

  • (UI) 좋아요버튼 테두리현상 해결 :
    -> 원인 = 탭인덱스. 헤일리와얘기해보고 탭인덱스 제거했습니다.

📢 논의하고 싶은 내용

리안 디테일페이지는 지하철역이슈없나요?

🎸 기타

@skiende74 skiende74 self-assigned this Oct 24, 2024
@skiende74 skiende74 changed the title Feat/884 jade fix editpage and focus and default [FE] UT피드백에의한 로직문제를 해결한다 Oct 24, 2024
Copy link

@ooherin ooherin changed the title [FE] UT피드백에의한 로직문제를 해결한다 [FE] UT피드백에 의한 로직 문제를 해결한다 Oct 24, 2024
Copy link

…ng-ggood into feat/884-jade-fix-editpage-and-focus-and-default
Copy link

@healim01 healim01 changed the title [FE] UT피드백에 의한 로직 문제를 해결한다 [FE] UT피드백에 의한 편집 페이지 새로고침 로직 문제를 해결한다 Oct 24, 2024
Copy link
Contributor

@healim01 healim01 left a comment

Choose a reason for hiding this comment

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

넵 확인했습니다~!

@healim01 healim01 merged commit 0c32ae8 into dev-fe Oct 24, 2024
3 checks passed
@healim01 healim01 deleted the feat/884-jade-fix-editpage-and-focus-and-default branch October 24, 2024 06:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants