Skip to content

Conversation

@plla2
Copy link
Member

@plla2 plla2 commented Feb 11, 2025

유형

  • 기능 구현
  • UI 구현
  • 리팩토링
  • 버그 해결
  • 문서 업데이트
  • 기타( )

작업 내용

  • MainHome 페이지 리디자인 적용

스크린샷

1234

6123

리뷰 요구사항

  • api호출 후 response 구조에 맞춰가기 위해서 데이터가 없을때의 NoContent 렌더링은 아직 리디자인 적용 안했습니다
  • 제철레시피 이미지들 중 ratio가 다른 몇몇 이미지의 크기를 고정시키다 보니 너비나 높이가 조금씩 늘어지거나 좁아지는 이미지들을 어떻게 하는게 좋을까요? (수정 완료)

@plla2 plla2 requested a review from grapefruit13 February 11, 2025 06:26
@plla2 plla2 self-assigned this Feb 11, 2025
Copy link
Contributor

@grapefruit13 grapefruit13 left a comment

Choose a reason for hiding this comment

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

Table 프롭 변경 말고는 대부분 스타일 수정인 것 같네요!
고생하셨어요!
픽셀값 관련해서 코멘트 하나 달았는데 한 번 확인해주십쇼~

className={cn('h-16 cursor-pointer odd:bg-grey-50 hover:bg-grey-100')}
className={cn(
'h-16 cursor-pointer odd:bg-grey-50 hover:bg-grey-100',
miniList ? 'h-[55.4px]' : '',
Copy link
Contributor

Choose a reason for hiding this comment

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

픽셀 값이 소수점 단위네요?! 유미님한테 사유를 한 번 여쭤보면 좋을 것 같아여

@plla2 plla2 merged commit f6acf66 into dev Feb 12, 2025
@plla2 plla2 deleted the THKV-131 branch February 12, 2025 03:32
plla2 added a commit to plla2/NNplanner-FE that referenced this pull request Feb 12, 2025
Feat[THKV-131]: MainHome 페이지 리디자인 적용
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants