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

safari에서 작동되지 않거나 의도한대로 보이지 않는 부분 수정 및 기타 이슈 #826

Merged
merged 4 commits into from
Oct 30, 2023

Conversation

Gilpop8663
Copy link
Collaborator

@Gilpop8663 Gilpop8663 commented Oct 26, 2023

🔥 연관 이슈

close: #823
close: #824
close: #825

📝 작업 요약

  • 사파리에서 이미지 확대 창 보이지 않는 부분 수정
    • 그러나 사파리에서 이미지 밖을 클릭했을 때 모달이 닫히지 않는데 닫기 버튼이 있어서 괜찮다고 생각했습니다
  • 사파리에서 로그인 버튼 이상하게 보이던 부분 수정
  • 테스트 코드 주석되어있던 것 복구
  • 게시글 작성 시 선택지가 카테고리를 가리던 것 수정

⏰ 소요 시간

30분

🔎 작업 상세 설명

개선 전

image image image

개선 후

image image image

@github-actions
Copy link

⚡️ Lighthouse report!

Category Score
🟠 Performance 62
🟠 Accessibilty 89
🟢 SEO 100
🟠 PWA 89
Category Score
🟢 First Contentful Paint 0.7 s
🟠 Largest Contentful Paint 3.5 s
🔴 Total Blocking Time 2,120 ms
🟢 Cumulative Layout Shift 0.046
🟢 Speed Index 3.2 s

Copy link
Member

@inyeong-kang inyeong-kang left a comment

Choose a reason for hiding this comment

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

사용성===우스

혹시 사파리에서 이미지 모달 밖 눌러도 닫히는 않는 것은 원인이 따로 있는지 궁금해요~!

// expect(result.current.getPageNumberList(totalPage)).toEqual(expected);
// }
// );

Copy link
Member

Choose a reason for hiding this comment

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

복구👍

@@ -14,6 +14,8 @@ export const Container = styled.div`
@media (min-width: ${theme.breakpoint.sm}) {
font: var(--text-body);
}

z-index: 1;
Copy link
Member

Choose a reason for hiding this comment

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

👍👍👍

@Gilpop8663
Copy link
Collaborator Author

사파리에서 모달이 닫히지 않는 이유로는 개발자 도구를 찍어보니 다이어로그 콘텐츠 안의 사각형을 위치를 계산하는 로직이 좀 다른 것 같아요

@jeomxon
Copy link
Collaborator

jeomxon commented Oct 26, 2023

이걸 오늘 해결해버리셨네요
당신은 LEGEND🔥

Copy link
Collaborator

@chsua chsua left a comment

Choose a reason for hiding this comment

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

정신없고 바쁜 와중에도 오류를 해결해버리는 우스는 최고..
우스의 열정에 반했습니다

사파리가 높이 계산하는게 이상해서 자꾸 뭐가 깨지는 것 같아요😂
배포하기전에 자주 확인해봐야겠네요

@chsua
Copy link
Collaborator

chsua commented Oct 29, 2023

fe-리뷰완

@Gilpop8663 Gilpop8663 merged commit 1d3a5e1 into dev Oct 30, 2023
2 checks passed
@woo-chang woo-chang deleted the feat/#824 branch December 6, 2023 07:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants