Skip to content

Conversation

@yyypearl
Copy link
Contributor

@yyypearl yyypearl commented Dec 18, 2024

연관 이슈

close #114


개요

UI 사용성 개선사항 반영

✅ 작업 내용

  • 내 궤도 메시지 > 새 편지함으로 기능명 수정
  • 행성 관리 특정 행성명 클릭시 해당 행성으로 이동
  • 편지 이미지 해상도 개선
  • 모바일 북마크 아이콘 적용
  • 16px 이하 input 확대 현상 수정
  • 행성 관리 페이지 스크롤 오류 수정

🖥 구현 결과

  • 화면 height 640px 이하 기기에서의 input 확대 현상 수정
KakaoTalk_20241218_183209401.mp4
  • 편지 이미지 해상도 개선
    KakaoTalk_20241218_183315405

  • 북마크 (홈화면 추가) 아이콘 적용
    image

  • 행성 관리 특정 행성명 클릭시 해당 행성으로 이동

20241218_183626.mp4
  • 행성 관리 페이지 스크롤 오류 수정
ScreenRecording_12-19-2024.16-00-05_1.mp4

리뷰 요구사항

이슈 사항 없습니다.

📝 기타 사항

  • 16px 이하에서의 input focus 확대 현상은 아래 블로그를 참고하여, 폰트를 기본 16px로 고정하고 scale을 사용해서 조정하였습니다. 이때, scale을 적용할 경우 폰트 자체의 크기만 작아지는 것이 아닌 input 영역이 모두 해당 배율만큼 줄어드는 문제가 있어, 줄어든 만큼의 여백을 확보하기 위해 width를 113%로 지정하고 편지 내용 작성 input 하단의 사진을 입력하는 부분은 position: absolute로 바로 아래 여백 없이 보여질 수 있도록 하였습니다! (그렇지 않을 경우, scale로 조정한 영역의 height는 고정이 되어 하단 여백이 남아 편지 내용 입력 항목과 사진 입력 항목 사이에 여백이 발생하였습니다!)
    [iOS/Safari] 의 focus 확대 방지하기

  • 이미지 해상도는 압축하면서 생기는 화질저하인듯한데, 우선 최대한 압축크기를 줄여보긴 했습니다. 모바일로 확인했을 때는 기존보다는 많이 좋아지긴 하였으나, 근본적으로는 서버에서 용량 제한이 해결이 되어야 할 것 같습니다!

  • 스크롤 오류의 경우 아래 블로그 참고했어요! 다른 사이트들 찾아보니까 스크롤시에 주소창이 아래로 내려가게 설정되어있더라구요. height: 100vh; 대신, height: auto; min-height: 100vh 적용해서 저도 그렇게 설정되도록 수정하였습니다. 알아두면 좋을 것 같아 공유해요!
    [React] 모바일 웹 스크롤 시 주소창, 하단바 없애기


@yyypearl yyypearl changed the title Feat/#114 [Feat] UI 사용성 개선사항 반영 Dec 18, 2024
@yyypearl yyypearl requested a review from hyo-4 December 18, 2024 09:41
@yyypearl yyypearl self-assigned this Dec 18, 2024
@yyypearl yyypearl added 💄 design 사용자 UI 및 CSS 파일 추가 · 수정 ✨ feat 새로운 기능 추가 labels Dec 18, 2024
@github-actions
Copy link

🎉 Deploy Preview

https://lettering-3xnxdd1lf-yyypearls-projects.vercel.app
여기에서 배포 결과 확인하기

@github-actions
Copy link

🎉 Deploy Preview

https://lettering-kjjb5ednm-yyypearls-projects.vercel.app
여기에서 배포 결과 확인하기

@hyo-4 hyo-4 merged commit 563a3a1 into develop Dec 20, 2024
1 check passed
@hyo-4
Copy link
Member

hyo-4 commented Dec 20, 2024

스크롤 오류 공유 고마워!! 확인확인
수고했어유 👍🙏🩷

name="viewport"
content="width=device-width, initial-scale=1, maximumScale=1, user-scalable=no"
/>
<link rel="apple-touch-icon" href="/favicon_114.png"></link>
Copy link
Member

Choose a reason for hiding this comment

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

이런 식으로 favicon 적용하는 군...

Copy link
Contributor Author

@yyypearl yyypearl Dec 20, 2024

Choose a reason for hiding this comment

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

응응 근데 북마크는 확인해보니 잘 적용이 안 된 거 같아ㅜ 다른 방법으로도 적용해볼게!

${theme.fonts.caption04}
}
} */
@media (max-height: 628px) {
Copy link
Member

Choose a reason for hiding this comment

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

따로 높이 지정해서 적용하는거 굳굳

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

✨ feat 새로운 기능 추가 💄 design 사용자 UI 및 CSS 파일 추가 · 수정

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feat] UI 사용성 개선사항 반영

3 participants