-
Notifications
You must be signed in to change notification settings - Fork 0
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
Conversation
🔗 배포된 storybook 주소: https://65f015a4aed45406385006ee-bkftfywlju.chromatic.com/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@@ -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'; |
There was a problem hiding this comment.
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> | ||
)} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
얘는 기능적으로 아무 역할 없이 그냥 존재하는거죠? 그냥 더 있다만 알려주는?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
피그마에 기능적인 얘기는 없더라구요 그래서 그냥 있다고만 알려주는 걸로 했슴다
@hae-on 아...태그 이름... 모르겠습니다 |
타미가 올려준 예시보면 + 버튼 width 이제 늘어나있는데 저거 한 사이즈로 고정 안해도 되나요? |
흠 근데 저거 저는 사이즈 고정보다 그냥 차있는게 더 나은거 같긴한데 어때요?? |
+를 고정하고 태그 길이를 늘이던가 하는게 나을까요? |
저는 개인적으로 +는 고정하고 태그 길이만 손보는 거 추천! |
@hae-on +를 피그마 폭으로 고정해버리면 이케 되는데 걍 글씨 안 보이게 할까요 |
아....저거 로컬 값은 또 길구나 진짜 에바쌈바네... |
앞에 이모티콘을 다 빼버릴가요 |
|
드디어 수정.! 확인 부탁 드림다. |
🔗 배포된 storybook 주소: https://65f015a4aed45406385006ee-pamwcsplsl.chromatic.com/ |
확인 완. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
굿굿~!!!
Issue
✨ 구현한 기능
📢 논의하고 싶은 내용
🎸 기타
⏰ 일정