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

feat: 오늘의 리뷰 태그 스타일 수정 #56

Merged
merged 9 commits into from
Apr 20, 2024
Merged

Conversation

Leejin-Yang
Copy link
Contributor

Issue

✨ 구현한 기능

  • 오늘의 리뷰 태그 스타일 수정

📢 논의하고 싶은 내용

  • x

🎸 기타

  • 2개만 보여주고 더 있다는 표시를 해줍니다
  • displaySlice 이름 잘못쓰고 있는 곳 수정했습니다.

⏰ 일정

  • 추정 시간 : 1시간
  • 걸린 시간 : 30분

Copy link

🔗 배포된 storybook 주소: https://65f015a4aed45406385006ee-bkftfywlju.chromatic.com/

Copy link
Contributor

@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.

황펭 수고하셨습니다.
정말 예전의 제가 한 짓이 기억나지 않네요.
기억 상실일까요?

스크린샷 2024-03-20 오후 12 41 40

제가 태그를 보니까 저희가 사용하는 것중에 제일 긴 태그가 '속이 풀려요'더라구요?
그거 집어넣으니까 + 버튼은 찌그러지고, 태그 안에 글자는 잘리네용
그때 얘기했던 것처럼 태그 안에 말줄임표는 별로여서 패쓰하구
그냥 태그 내부 글자를 4글자 이하로 다 맞출까요?
그게 젤 속편하긴 한데,,,

@@ -7,7 +7,7 @@ import { RecipeItem } from '@/components/Recipe';
import { PATH } from '@/constants/path';
import { useIntersectionObserver } from '@/hooks/common';
import { useInfiniteMemberRecipeQuery } from '@/hooks/queries/members';
import useDisplaySlice from '@/utils/displaySlice';
import displaySlice from '@/utils/displaySlice';
Copy link
Contributor

Choose a reason for hiding this comment

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

기억이 안나는데 왜 커스텀 훅도 아닌데 useDisplaySlice라고 쓰고 있었던거죠?? 저유틸 내가 만든건데???

<li className={cx(tag, tagExtra)}>
<span>+</span>
</li>
)}
Copy link
Contributor

Choose a reason for hiding this comment

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

얘는 기능적으로 아무 역할 없이 그냥 존재하는거죠? 그냥 더 있다만 알려주는?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

피그마에 기능적인 얘기는 없더라구요 그래서 그냥 있다고만 알려주는 걸로 했슴다

@Leejin-Yang
Copy link
Contributor Author

@hae-on 아...태그 이름... 모르겠습니다

@xodms0309
Copy link
Member

스크린샷 2024-03-20 오후 4 57 04

지금 시안을 보면 오늘의 리뷰에는 태그 이름에 이모티콘이 붙어있고 검색페이지에는 안 붙어있는데 어차피 태그 이름을 다 통일해야해서 그거 부터 결정되고 생각해봅시다. 만약 이모티콘 안붙으면 상관 없을듯해요

@hae-on
Copy link
Contributor

hae-on commented Mar 20, 2024

타미가 올려준 예시보면 + 버튼 width 이제 늘어나있는데 저거 한 사이즈로 고정 안해도 되나요?
앞에 태그 길이에 따라서 + 버튼도 같이 늘어났다 줄어났다하는거 같은데??

@xodms0309
Copy link
Member

흠 근데 저거 저는 사이즈 고정보다 그냥 차있는게 더 나은거 같긴한데 어때요??

@Leejin-Yang
Copy link
Contributor Author

@xodms0309 @hae-on

+를 고정하고 태그 길이를 늘이던가 하는게 나을까요?

@hae-on
Copy link
Contributor

hae-on commented Mar 20, 2024

저는 개인적으로 +는 고정하고 태그 길이만 손보는 거 추천!
길이만 늘려도 되고, 아님 패딩을 손봐도 괜찮을 듯??
황펭이 보기에 자연스러운 길이로 해주시져

@Leejin-Yang
Copy link
Contributor Author

스크린샷 2024-03-20 23 30 35

@hae-on +를 피그마 폭으로 고정해버리면 이케 되는데 걍 글씨 안 보이게 할까요

@hae-on
Copy link
Contributor

hae-on commented Mar 20, 2024

아....저거 로컬 값은 또 길구나 진짜 에바쌈바네...
그냥 안보이게 하시져 제가 보기엔 진짜 태그 길이 4자 조정 가야할 듯?

@xodms0309
Copy link
Member

앞에 이모티콘을 다 빼버릴가요

@Leejin-Yang
Copy link
Contributor Author

Leejin-Yang commented Mar 20, 2024

# 으로 통일하죠.?

@Leejin-Yang
Copy link
Contributor Author

@xodms0309 @hae-on

스크린샷 2024-04-20 00 42 39

드디어 수정.! 확인 부탁 드림다.
피그마 스타일 수정된 것도 했어요

Copy link

🔗 배포된 storybook 주소: https://65f015a4aed45406385006ee-pamwcsplsl.chromatic.com/

@hae-on
Copy link
Contributor

hae-on commented Apr 20, 2024

확인 완.
수고했어요~~~!

Copy link
Member

@xodms0309 xodms0309 left a comment

Choose a reason for hiding this comment

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

굿굿~!!!

@Leejin-Yang Leejin-Yang merged commit fd21bc7 into feat/v2 Apr 20, 2024
2 of 3 checks passed
@Leejin-Yang Leejin-Yang deleted the feat/issue-54 branch April 20, 2024 07:08
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.

오늘의 리뷰 태그 스타일 수정
3 participants