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

refactor: 중복 디자인 수정 #167

Merged
merged 5 commits into from
Jul 27, 2024
Merged

refactor: 중복 디자인 수정 #167

merged 5 commits into from
Jul 27, 2024

Conversation

hae-on
Copy link
Contributor

@hae-on hae-on commented Jul 24, 2024

Issue

✨ 구현한 기능

1. 카테고리 이미지 변경

스크린샷 2024-07-24 오후 10 32 27 스크린샷 2024-07-24 오후 10 32 43

원래 s3에 올라가있는 이미지가 52x52인데 56x56으로 교체됐습니다.

원래는 하나의 디자인으로 통일하려고 했는데
얼추 맞는 거 같아서, 그냥 뒀습니다!
식품이랑 음료가 조금 삐뚤하긴 한데 혹시 거슬리면 말해주세요.
네모로 통일해버리게!

2. 좋아요 버튼 수정

그림자가 필요해 디자이너분이 말씀하신대로 svg에서 png로 교체하였습니다.
이제 흰 배경에서도 잘 보여요!

스크린샷 2024-07-24 오후 10 37 42

📢 논의하고 싶은 내용

우리 레시피 상세 들어가려면 로그인 해야 되더라구요.
이거 없애기로 하지 않았나요??

🎸 기타

x

⏰ 일정

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

Copy link

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

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.

훨씬 낫네용. 리뷰는 확실히 로그인 없애기로 했는데 꿀조합도 없애기로 했었나요?!??

Copy link
Contributor

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

레시피 상세도 로그인 없애나요.?
근데 그게 맞는듯요? Post 요청만 로그인 받고 (댓글, 리뷰, 좋아요 등)
수고했어요 👍

@hae-on
Copy link
Contributor Author

hae-on commented Jul 27, 2024

얍~ 조회하는 건 로그인 없는게 사용자한테는 더 좋을 거 같아요!
그럼 다음 이슈에서 레시피 상세 로그인 없앨게욥!

@hae-on hae-on merged commit 411f5e3 into feat/v2 Jul 27, 2024
2 of 3 checks passed
@hae-on hae-on deleted the refactor/issue-166 branch July 27, 2024 06:14
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