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

design: 결과 페이지 퍼블리싱 #53

Merged
merged 8 commits into from
Feb 11, 2025
Merged

Conversation

xxxjinn
Copy link
Collaborator

@xxxjinn xxxjinn commented Feb 11, 2025

⚾️ Related Issues

📝 Task Details

  • 하림님이 맡으신 보따리 상세 페이지 퍼블리싱 hifi에 맞게 약간 수정했습니다!

    • 보따리 이름 폰트 적용
    • 보따리 선물 리스트 사진들 gap 수정 (MyCardList.tsx) gap-4 -> gap-[12px]
  • 보따리 결과 페이지 퍼블리싱을 했습니다.

  • Card.tsx 컴포넌트에 active와 hover 관련한 선택적 props (noHover, noActive)를 추가했습니다. (해당 페이지에서는 이미지에 hover해도 아무 일이 안일어나도록 하기 위해서!)

📂 References

  • name의 길이가 너무 길어지면 ellipse가 보이도록 해두었습니다. (두 번째 선물 참고)
  • 링크 바로가기 관련해서는, 해당 부분을 button variant="link"를 사용했기 때문에 해당 버튼에 hover 시 밑줄이 나오게 됩니다.
    • 링크가 없는 버튼은 색을 연하게 해뒀는데, 구분이 잘 안가는 것 같아서 피그마에 문의드려야 할 것 같아요.
image
  • 목록 각 요소 hover 시 opacity-70주고, 각 요소 cursor-pointer로 일단 해놨습니다! (꼭 우측 화살표를 눌러서 이동하는 것이 아닌, 각 요소를 눌렀을 때 선물 상세 페이지로 이동할 수도 있다고 생각해서요) => 첨부드린 사진은 포인터가 안나와있는데, 맨 첫 번째 요소에 hover했을 때 입니다.
image

💕 Review Requirements

  • hifi가 어떻게 나올지 모르는 상태에서 저희가 역할을 분담했었어서,,, 뭔가 애매하게 (?) 제가 맡아버렸네요
  • mockData를 통해 해당 페이지 내부에서 진행했습니다.
  • header에 bg-white를 해야할 것 같은데 어떻게 생각하시나용?.?

@xxxjinn xxxjinn added the 🎨 Html & CSS 마크업 & 스타일링 label Feb 11, 2025
@xxxjinn xxxjinn self-assigned this Feb 11, 2025
@xxxjinn xxxjinn marked this pull request as ready for review February 11, 2025 12:32
Copy link
Member

Choose a reason for hiding this comment

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

여기 /giftbag/detail/[id]/answer 제가 설정해둔 url,, 약간 애매한 것 같아서 /detail/[id] 형식말고 /list/[id] 이런 식으로 수정하는게 더 직관적일 것 같은데 어떻게 생각하세요??

현재는 /giftbag/list에서 giftbag/detail/[id] 로 이동되는데, /giftbag/list/[id]로 이동되게끔요!

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

저두 detail보다는 list가 직관적이고 좋을 것 같아요!

Copy link
Member

@gkfla668 gkfla668 Feb 13, 2025

Choose a reason for hiding this comment

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

여기 /giftbag/detail/[id]/answer 제가 설정해둔 url,, 약간 애매한 것 같아서 /detail/[id] 형식말고 /list/[id] 이런 식으로 수정하는게 더 직관적일 것 같은데 어떻게 생각하세요??

현재는 /giftbag/list에서 giftbag/detail/[id] 로 이동되는데, /giftbag/list/[id]로 이동되게끔요!

추후에 한꺼번에 수정해놓을게요 ! !

@gkfla668
Copy link
Member

gkfla668 commented Feb 11, 2025

header에 bg-white를 해야할 것 같은데 어떻게 생각하시나용?.?

아 header에 지금 아무 색상도 안들어가있죠?!
색상 안넣으면 어디서 어떻게 보이나요..?!

@gkfla668
Copy link
Member

링크가 없는 버튼은 색을 연하게 해뒀는데, 구분이 잘 안가는 것 같아서 피그마에 문의드려야 할 것 같아요.

고러게요..! 문의드려볼게용

@gkfla668
Copy link
Member

하림님이 맡으신 보따리 상세 페이지 퍼블리싱 hifi에 맞게 약간 수정했습니다!

보따리 이름 폰트 적용
보따리 선물 리스트 사진들 gap 수정 (MyCardList.tsx) gap-4 -> gap-[12px]

오옹 감사합니다!

고생 많으셨어요~~! 👍

@xxxjinn
Copy link
Collaborator Author

xxxjinn commented Feb 11, 2025

header에 bg-white를 해야할 것 같은데 어떻게 생각하시나용?.?

아 header에 지금 아무 색상도 안들어가있죠?! 색상 안넣으면 어디서 어떻게 보이나요..?!

고냥 투명하게 되어있어서 내부 컨텐츠가 다 비치는... 그렇게 보이더라구용

@xxxjinn xxxjinn merged commit b0cb980 into dev Feb 11, 2025
1 check passed
@xxxjinn xxxjinn deleted the style/result-publishing branch February 11, 2025 14:53
Copy link
Member

@gkfla668 gkfla668 Feb 13, 2025

Choose a reason for hiding this comment

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

여기 /giftbag/detail/[id]/answer 제가 설정해둔 url,, 약간 애매한 것 같아서 /detail/[id] 형식말고 /list/[id] 이런 식으로 수정하는게 더 직관적일 것 같은데 어떻게 생각하세요??

현재는 /giftbag/list에서 giftbag/detail/[id] 로 이동되는데, /giftbag/list/[id]로 이동되게끔요!

추후에 한꺼번에 수정해놓을게요 ! !

Comment on lines +90 to +155
<div className="px-4 py-6">
{Object.entries(groupedGifts).map(([message, gifts], idx) => (
<div key={message} className="flex flex-col gap-[26px]">
<div>
{idx !== 0 && (
<hr className="my-4 border-gray-100 border-[1px] mb-[26px]" />
)}
<AnswerChip text={message} />
</div>
<div className="flex flex-col gap-[14px]">
{gifts.map((gift, index) => (
<div
className="flex items-center h-[70px] justify-between relative hover:opacity-70 cursor-pointer"
key={index}
>
<div className="flex gap-4 w-full">
<Card
img={gift.images[0]}
size="small"
type="image"
noHoverStyle={true}
noActiveStyle={true}
/>
<div className="flex justify-center flex-col items-start">
<div className="text-[15px] font-medium max-w-[250px] overflow-hidden text-ellipsis whitespace-nowrap">
{gift.name}
</div>
{gift.link ? (
<Link
href={gift.link}
target="_blank"
rel="noopener noreferrer"
>
<Button
variant="link"
className="text-xs text-gray-400"
>
링크 바로가기
</Button>
</Link>
) : (
<div>
<Button
variant="link"
disabled
className="text-xs text-gray-200"
>
링크 바로가기
</Button>
</div>
)}
</div>
</div>
<Button
variant="ghost"
className="absolute left-44 top-1/2 transform -translate-y-1/2"
>
<Image src={ArrowIcon} alt="arrowIcon" />
</Button>
</div>
))}
</div>
</div>
))}
</div>
);
Copy link
Member

@gkfla668 gkfla668 Feb 13, 2025

Choose a reason for hiding this comment

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

요것도 그 header 투명하게 비친다고 하신거
image
header에 bg-white로 해주면 될 것 같은데, 전체가 스크롤 되는 현상이 있어서

내부 부분에 height 설정하고 scroll 적용하고 새로 pr올리겠습니다.!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🎨 Html & CSS 마크업 & 스타일링
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Style] 결과 페이지 퍼블리싱
2 participants