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

모달에서 ScrollSnapContainer의 스와이프가 동작하는 오류 수정 #517

Merged
merged 6 commits into from
Oct 5, 2023

Conversation

solo5star
Copy link
Member

#️⃣ 연관된 이슈

close #516

📝 작업 내용

  • useScrollSnapGuard 를 구현하여 ScrollSnapContainer에서 사용되는 이벤트들의 propagation을 중단할 수 있도록 해주었습니다.
  • 모달에 useScrollSnapGuard를 사용하면 모달 스크롤 시 카페 목록이 스와이프되는 버그를 해결할 수 있습니다.

💬 리뷰 요구사항

@solo5star solo5star added 카테고리:버그👾 고쳐줘잉 FE 될 때까지 한다잉 FE:홈화면 홈화면과 관련된 이슈입니다 우선순위:🟠높음 비교적 먼저 작업해야 합니다. labels Sep 29, 2023
@solo5star solo5star self-assigned this Sep 29, 2023
Copy link
Collaborator

@jeongwusi jeongwusi left a comment

Choose a reason for hiding this comment

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

나이스 코드~

*/
const useScrollSnapGuard = () => {
const preventPropagation: EventHandler<SyntheticEvent> = useCallback((event) => {
event.stopPropagation();
Copy link
Collaborator

Choose a reason for hiding this comment

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

ㅋㅋㅋㅋ event.stopPropagation()으로 모달에서 스와이프가 동작하는 오류를 수정하셨군여!

좋습니당

@solo5star solo5star merged commit a669efa into dev Oct 5, 2023
1 check failed
@solo5star solo5star deleted the fix/516-prevent-touch-swipe-on-modal branch October 5, 2023 05:41
@solo5star solo5star restored the fix/516-prevent-touch-swipe-on-modal branch October 5, 2023 05:41
@solo5star solo5star deleted the fix/516-prevent-touch-swipe-on-modal branch October 5, 2023 05:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
FE 될 때까지 한다잉 FE:홈화면 홈화면과 관련된 이슈입니다 우선순위:🟠높음 비교적 먼저 작업해야 합니다. 카테고리:버그👾 고쳐줘잉
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

모달이 열린 상태에서 터치 스와이프가 동작하고 있다
3 participants