Skip to content

Conversation

@junjeeong
Copy link
Collaborator

@junjeeong junjeeong commented Nov 6, 2024

preview

스크린샷 2024-11-06 오후 9 59 25

설명

  • 전해주어야 하는 인자(info) 의 타입은 아래와 같습니다.
스크린샷 2024-11-06 오후 10 00 17
  • return문이 너무 길어서 추상화가 더 필요해보입니다. 리팩토링은 추후에 하겠습니다.

@mungyun
Copy link
Contributor

mungyun commented Nov 7, 2024

타입오류 뜹니다!

@junjeeong
Copy link
Collaborator Author

타입오류 뜹니다!

고쳤습니다!

<section className="relative w-full h-[60%]">
<Image
src={info.imageSource || `/images/no-content.svg`}
objectFit="cover"
Copy link
Collaborator

Choose a reason for hiding this comment

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

objectFit="cover" 이거 현재 버전에서 경고 뜰거에요
style={{ objectFit: 'cover' }} 이렇게 바꾸면 👍

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

옹 어떤 경고죠?

<section className="w-full h-[40%] flex flex-col justify-between gap-[10px] pt-[15px] px-[20px] pb-[10px]">
<div className="flex justify-between">
<span className="text-sm text-gray-400">
{createdTime || "1일 전"}
Copy link
Collaborator

Choose a reason for hiding this comment

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

작성시간 계산하는거는 나중에 따로 구현하실건가욤?

Copy link
Collaborator

Choose a reason for hiding this comment

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

util/timeAgo를 제가 못봤었네요 ㅎ

<Image src="/icons/kebab.svg" alt="kebab button" fill />
</div>
</div>
<div className="text-[black100] text-lg ">
Copy link
Collaborator

Choose a reason for hiding this comment

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

저는 text-black100 으로 작성해도 정상적으로 적용되더라구요!

Copy link
Collaborator

@hongggyelim hongggyelim left a comment

Choose a reason for hiding this comment

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

수고하셨습니다

@junjeeong junjeeong merged commit 925edfb into develop Nov 7, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants