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

fix: 모바일에서 충전소 리스트가 열리지 않는 오류 수정 #667

Merged
merged 1 commit into from
Sep 8, 2023
Merged

fix: 모바일에서 충전소 리스트가 열리지 않는 오류 수정 #667

merged 1 commit into from
Sep 8, 2023

Conversation

kyw0716
Copy link
Member

@kyw0716 kyw0716 commented Sep 8, 2023 •
edited by github-actions bot
Loading

📄 Summary

StationList가 좌측 패널로 사용될 때 작성했던 코드의 잔재가 남아있어 그 부분 때문에 발생한 오류였습니다.

오류 발생 원인

  • StationList가 좌측 패널로 사용될 때 충전소를 선택하면 StationList를 강제로 닫아버리는 로직이 존재했음
  • 네비게이터에서 StationList가 패널 메뉴에서 사라짐
  • 네비게이터에서 사라진 이후 모바일 대응을 하다가 StationList가 다시 모바일에서 사용됨
  • 강제로 닫아버리는 로직 때문에 마커를 클릭한 이후 StationList를 열려고 시도하면 강제로 닫아버리는 로직에 의해 리스트가 닫히는 버그가 생김

해결 방법

  • StationList를 강제로 닫아버리는 로직은 이제 사용하지 않기 때문에 이 로직을 수행하던 부분을 지워줌

🕰️ Actual Time of Completion

30분

🙋🏻 More

close #666

@kyw0716 kyw0716 added 🚑 버그 예기치 않은 문제 또는 의도하지 않은 동작입니다 FE 프론트엔드 관련 이슈입니다 🚨 긴급 제일 최우선으로 해결해야 할 사항입니다 labels Sep 8, 2023
@kyw0716 kyw0716 self-assigned this Sep 8, 2023
@kyw0716 kyw0716 temporarily deployed to test September 8, 2023 07:57 — with GitHub Actions Inactive
@github-actions
Copy link

github-actions bot commented Sep 8, 2023

🚀storybook: https://storybook.carffe.in/

feb-dain
feb-dain approved these changes Sep 8, 2023
View reviewed changes
Copy link
Collaborator

@feb-dain feb-dain left a comment

Choose a reason for hiding this comment

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

굿

@kyw0716 kyw0716 merged commit 3a6a1b9 into develop Sep 8, 2023
4 checks passed
@kyw0716 kyw0716 deleted the fix/666 branch September 8, 2023 10:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gabrielyoon7 gabrielyoon7 gabrielyoon7 approved these changes

@feb-dain feb-dain feb-dain approved these changes

Assignees

@kyw0716 kyw0716

Labels
FE 프론트엔드 관련 이슈입니다 🚑 버그 예기치 않은 문제 또는 의도하지 않은 동작입니다 🚨 긴급 제일 최우선으로 해결해야 할 사항입니다
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

모바일에서 충전소 리스트가 열리지 않는 오류를 개선한다
3 participants
@kyw0716 @gabrielyoon7 @feb-dain