-
Notifications
You must be signed in to change notification settings - Fork 4
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
Conversation
⚡️ Lighthouse report!
|
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.
사용성===우스
혹시 사파리에서 이미지 모달 밖 눌러도 닫히는 않는 것은 원인이 따로 있는지 궁금해요~!
// expect(result.current.getPageNumberList(totalPage)).toEqual(expected); | ||
// } | ||
// ); | ||
|
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.
복구👍
@@ -14,6 +14,8 @@ export const Container = styled.div` | |||
@media (min-width: ${theme.breakpoint.sm}) { | |||
font: var(--text-body); | |||
} | |||
|
|||
z-index: 1; |
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.
정신없고 바쁜 와중에도 오류를 해결해버리는 우스는 최고..
우스의 열정에 반했습니다
사파리가 높이 계산하는게 이상해서 자꾸 뭐가 깨지는 것 같아요😂
배포하기전에 자주 확인해봐야겠네요
fe-리뷰완 |
🔥 연관 이슈
close: #823
close: #824
close: #825
📝 작업 요약
⏰ 소요 시간
30분
🔎 작업 상세 설명
개선 전
개선 후