Skip to content

Conversation

@crazyupinc-design
Copy link
Collaborator

@crazyupinc-design crazyupinc-design commented Dec 29, 2024

이슈 번호

close #280

변경 사항 요약

  • 메인 이미지들 너무깨져보여서 2x 으로 받아서 확인해봐야할듯
    => 이미지 모두 2x로 교체했습니다

  • 로그인/회원가입/계정설정: 모바일 양옆 패딩안잡혀있어서 375이하는 간격없이 줄어듬
    => 모바일일때 좌우패딩 20px설정했습니다

  • 접근성 관련 에러 입니다 input에 autocomplete="current-password" 추가 해주셔야할듯합니다( 패스워드에서만)
    => autocomplete="current-password" 추가했습니다

  • 메인랜딩 에니메이션 작업했습니다
    => framer-motion 사용함(npm install 명령어로 설치하세요), 이미지 스크롤은 CSS 애니메이션 사용했습니다

Doc

컴포넌트 인 경우 props를 입력해주세요

Props Type Description
내용 입력
내용 입력

hooks 및 utils 인 경우 input,output을 입력해주세요

prams Type Description
input 내용 입력
output 내용 입력

테스트 결과

베이스(develop) 브랜치에 포함되기 위한 코드는 모두 정상적으로 작동이 되어야 합니다.
컴포넌트의 경우 스크린샷을 포함해주세요.

@crazyupinc-design crazyupinc-design added ✨ 신규 기능 신규 기능 작업 관련 태그 🐛 버그 버그 관련 태그 💄 스타일 스타일 작업 태그 labels Dec 29, 2024
@crazyupinc-design crazyupinc-design self-assigned this Dec 29, 2024
Copy link
Contributor

@junghwaYang junghwaYang left a comment

Choose a reason for hiding this comment

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

image 여기가 모션 진행될때 겹치는것같아요 전체적인 인터럭션 좋은것같습니다 👍🏻
hook.js:608 Image with src "/images/img-write.png" has either width or height modified, but not the other. If you use CSS to change the size of your image, also include the styles 'width: "auto"' or 'height: "auto"' to maintain the aspect ratio

콘솔 에러 확인 부탁드립니다~

계정설정에서 320으로 봤을때 패딩이 안들어오는것같습니다
input 고정값때문인거같은데 다른 로그인 로그아웃처럼 유연하게 동작할수있게 해주세요

Copy link
Collaborator

@haksoo0918 haksoo0918 left a comment

Choose a reason for hiding this comment

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

라이브러리를 사용하셔서 애니메이션을 적용하셨군요.
새로운걸 적용하시려면 또 공부하셨어야 했을텐데 대단하세요. 👍

@crazyupinc-design
Copy link
Collaborator Author

image 여기가 모션 진행될때 겹치는것같아요 전체적인 인터럭션 좋은것같습니다 👍🏻

hook.js:608 Image with src "/images/img-write.png" has either width or height modified, but not the other. If you use CSS to change the size of your image, also include the styles 'width: "auto"' or 'height: "auto"' to maintain the aspect ratio

콘솔 에러 확인 부탁드립니다~

계정설정에서 320으로 봤을때 패딩이 안들어오는것같습니다 input 고정값때문인거같은데 다른 로그인 로그아웃처럼 유연하게 동작할수있게 해주세요

  1. 해당 이슈(모션 겹치는부분)가 제환경에서는 재현되지 않고 있습니다. ㅜㅜ
  2. 콘솔 에러 : 코드 수정했습니다. (모바일과 태블릿에서 width와 height 모두 지정)
  3. 로그인, 회원가입과 동일하게 처리했습니다. 320일때패딩적용됩니다

@junghwaYang
Copy link
Contributor

종모양 관련해서는 제 캐싱 버그엿나봐요 계정설정은 확인되었고
/images/img-text-white.png 해당 이미지에 대한 동일한 콘솔 경고가 뜹니다 확인해주세요!

@crazyupinc-design
Copy link
Collaborator Author

콘솔

image 수정했습니다 오류 안뜨고 있습니다

Copy link
Collaborator

@solprime solprime left a comment

Choose a reason for hiding this comment

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

오류 안뜨네요! 수고하셨어요~

@junghwaYang junghwaYang merged commit 5a40d6a into develop Dec 30, 2024
1 check passed
@junghwaYang junghwaYang deleted the feature/#280_QA-반영---메인이미지-모바일 branch December 30, 2024 11:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

✨ 신규 기능 신규 기능 작업 관련 태그 🐛 버그 버그 관련 태그 💄 스타일 스타일 작업 태그

Projects

Status: 완료

Development

Successfully merging this pull request may close these issues.

QA 반영 - 메인이미지, 모바일

5 participants