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

[FE] feat: Skeleton UI 추가 #684

Merged
merged 8 commits into from
Sep 21, 2023
Merged

[FE] feat: Skeleton UI 추가 #684

merged 8 commits into from
Sep 21, 2023

Conversation

xodms0309
Copy link
Collaborator

Issue

✨ 구현한 기능

  • 이미지에 대해서 스켈레톤 ui 적용
  • 이미지 구겨보이는 것들 object-fit: cover 속성 적용, 배너 이미지 높이 추가
  • 사용안하는 파일 삭제

📢 논의하고 싶은 내용

x

🎸 기타

x

⏰ 일정

  • 추정 시간 :
  • 걸린 시간 : 2시간

@github-actions
Copy link

github-actions bot commented Sep 20, 2023

Test Results

8 tests   8 ✔️  7s ⏱️
3 suites  0 💤
1 files    0

Results for commit 74feca1.

♻️ This comment has been updated with latest results.

Copy link
Collaborator

@Leejin-Yang Leejin-Yang left a comment

Choose a reason for hiding this comment

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

코멘트 달아둔 부분 수정해주세요~
추가적으로 이미지에 loading='lazy'도 추가 부탁.!

@@ -81,6 +81,7 @@ export default HomePage;

const Banner = styled.img`
width: 100%;
height: auto;
Copy link
Collaborator

Choose a reason for hiding this comment

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

Banner 컴포넌트에 height 추가해주세요~
360입니다

Copy link
Collaborator

@hae-on hae-on left a comment

Choose a reason for hiding this comment

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

잘 하셨네여 👏

Copy link
Collaborator

@Leejin-Yang Leejin-Yang left a comment

Choose a reason for hiding this comment

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

코멘트 하나만 확인 부탁!

Comment on lines 22 to 28
<ProductImage
src={image}
width={90}
height={90}
alt={`${name}사진`}
onLoad={() => setIsImageLoading(false)}
/>
Copy link
Collaborator

Choose a reason for hiding this comment

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

리스트에 쓰이는 이미지에 loading='lazy' 넣어주세요.! 배너는 지연 로딩 없어도 될듯 맨위에 있어서

@xodms0309 xodms0309 merged commit 762ba8c into develop Sep 21, 2023
1 check passed
@xodms0309 xodms0309 deleted the feat/issue-576 branch September 21, 2023 10:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FE] Skeleton UI 추가
3 participants