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: RecommendList 컴포넌트 리팩토링 #78

Merged
merged 4 commits into from
Apr 9, 2024

Conversation

xodms0309
Copy link
Member

Issue

✨ 구현한 기능

스크린샷 2024-04-07 오후 3 30 10
  • RecommendList의 바뀐 디자인 적용

📢 논의하고 싶은 내용

  • 논의하고 싶은 내용을 작성합니다.

🎸 기타

  • �원래 저 '상품' 텍스트가 상품/꿀조합이었는데 현재 자동완성이 상품만 되어서 텍스트 고정해놨습니다 (피그마 참고)

⏰ 일정

  • 추정 시간 : 2
  • 걸린 시간 : 1

Copy link

github-actions bot commented Apr 7, 2024

🔗 배포된 storybook 주소: https://65f015a4aed45406385006ee-vtqrmzohfy.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.

굿굿 수고했습니다 타미~

'상품' 고정 해놓은 건 나중에 기능 추가되면 바뀌는건가요?

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.

수고했어요 타미.!
궁금한거 하나 질문 달았어요

Comment on lines +22 to +24
<Text as="span" weight="semiBold">
{fragment}
</Text>
Copy link
Contributor

Choose a reason for hiding this comment

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

이렇게 하면 <span>아아</span><mark>이이</mark><span>오오</span> 으로 되는건가요? 정규식을 잘 몰라서

Copy link
Member Author

Choose a reason for hiding this comment

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

이거 황펭이 구현했던거자나요
댓글 내용이 맞습니당

@xodms0309
Copy link
Member Author

@hae-on 넹 근데 이거 자동완성은 백엔드의 영역이라서..! 추가되려면 백엔드와 논의가 필요해보입니다

@hae-on
Copy link
Contributor

hae-on commented Apr 9, 2024

바로 머지하시죠~~

@xodms0309 xodms0309 merged commit 0c61323 into feat/v2 Apr 9, 2024
2 of 3 checks passed
@xodms0309 xodms0309 deleted the feat/issue-65 branch April 9, 2024 14:22
@Creative-Lee
Copy link

너무 기대가 됩니다 얼른 배포해주세요

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.

RecommendList 컴포넌트 리팩토링
4 participants