refactor: 디자인 2차 수정 #165
Open
refactor: 디자인 2차 수정 #165
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
🔗 배포된 storybook 주소: https://65f015a4aed45406385006ee-kwkqeuebdw.chromatic.com/ |
xodms0309
reviewed
Jul 26, 2024
| @@ -11,9 +11,9 @@ export const container = style({ | |||
| export const commentForm = style({ | |||
| display: 'flex', | |||
| gap: 8, | |||
| alignItems: 'flex-end', | |||
| alignItems: 'center', | |||
There was a problem hiding this comment.
이거 근데 댓글이 길어지면 어떻게 보이나요? 저 이거 작업할 때 꽤나 골머리 앓았던 기억이 있어가지고
There was a problem hiding this comment.
그래서 flex-end 썼었구나,,,
얘 이렇게 보이네요 길어지면 밑에 붙게 함 해볼게욥
There was a problem hiding this comment.
Leejin-Yang
approved these changes
Jul 26, 2024
| @@ -13,6 +13,7 @@ export const productImage = style({ | |||
| width: '100%', | |||
| height: 'auto', | |||
| minWidth: 163, | |||
| borderRadius: '6px', | |||
There was a problem hiding this comment.
Suggested change
| borderRadius: '6px', | |
| borderRadius: 6, |
..!
| export const wrapper = style({ | ||
| height: '100%', | ||
| display: 'flex', | ||
| flex: '0 0 160px', |
There was a problem hiding this comment.
프론트엔드 시니어와 함께 한 CSS랄까요~~
|
저도 집가면 로컬 확인하겠슴다 |
|
원래 로컬에서 돌리면 oauth 진입 안했던가요?? 지금 여기 진입이 안돼서 로그인이 안되고 mockSessionId 설정이 안되는듯 |
|
지금 로컬은 무조건 로그인 상태 아녜여? 그 내가 레시피 상세 msw 주석 처리한 거 때문에 그런가?? 아직도 안 들어가져요??? |
|
로컬 무조건 로그인 되어있슴다 |
|
🔗 배포된 storybook 주소: https://65f015a4aed45406385006ee-siirvvwepk.chromatic.com/ |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Issue
✨ 구현한 기능
1. 좋아요 카운트 있을 때만 gap 추가
요 이슈 해결했습니다.
레시피 좋아요가 1. 꿀조합 컴포넌트 2. 꿀조합 상세 에서 사용되고 있는데,
gap은 좋아요 카운트가 있는 2. 꿀조합 상세에만 필요하기 때문에
어쩔 수 없이 조건부 스타일을 주게 되었습니다.
2. textarea 상하좌우 동일하게 수정
요 이슈 해결했습니다.
이게 제 생각에는 전송 버튼이 문제라기보다,,,
저 글자 수를 나타내는 글자가 수평이 아니고, textarea 자체 위 아래 마진이 안 맞아서 나타나는 문제 같아요.
그래서 원래는 요랬는데,
요래 됐슴다.
3. 하나일 때 커지는 현상
요 이슈 해결했습니다.
이제 하나일 때도, 여러개 일때도 같은 사이즈 입니다!
검색창 부분도 똑같은 문제라 동일한 방법으로 해결했는데,
로컬에서는 볼 방법이 없어서 (⬇️ 이렇기 때문)
걍 배포하고 확인하시죵 >_<
4. 상품 이미지 border 값 추가
이 말이 상품 이미지도 다른 애들처럼 border 값 넣으라는 거 맞겠죠??
요런 애들이 '상품 이미지가 테두리까지 꽉 차있는 경우' 라는 거죠?
나는 이렇게 이해해서 border 값 추가했음!
📢 논의하고 싶은 내용
엑셀에 처리한 건 색칠해놨어요.
일단 제가 할 수 있는 부분은 다 한 듯....
(밑에 두개랑 톡방에 올려놓은 건 제가 할게용,,,)
나머지는 도와줘요 여러분.... 🥹
지금 시급한 것 레시피 좋아요 버튼!!!!!!!!!
🎸 기타
아니 로컬에서 레시피 상세 어캐 들어감??
진짜로 아이디 1부터 계속 넣어봤는데 우리 따로 설정을 안해뒀나요...?
http://localhost:3000/recipes/1
⏰ 일정