Skip to content

Conversation

@plla2
Copy link
Member

@plla2 plla2 commented Mar 25, 2025

유형

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

작업 내용

  • 랜딩페이지 작업

리뷰 요구사항

  • npm install 한번 해주십셔
  • env는 디코를 통해서 전달드렸습니다! 꼭 넣고 돌려주십셩
  • 문의 메일보내기 스키마는 정확하게 정해서 안지켜질 때의 예외처리도 유미님께 부탁드려야겠어여
  • 제일 상단으로 보내주는 top 플로팅버튼 디자인도 부탁드려서 만드는게 좋아보여요!
  • 이메일 정상적으로 보내고 toast뜰때 success 배경색과 랜딩페이지의 배경색과 같아서 안보이는데 이것도 말씀드릴게여
  • 스크린샷은 페이지도 많고, 애니메이션이 있어서 pull받아서 직접 돌려보시는 게 좋을 것 같아 안올릴게영

plla2 added 30 commits March 24, 2025 12:45
@plla2 plla2 requested a review from grapefruit13 March 25, 2025 05:58
@plla2 plla2 self-assigned this Mar 25, 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.

고생하셨습니다!! 랜딩 너무 멋있게 잘 만드셨네여~~

코멘트 몇 개 달았는데 확인 부탁드립니다!

@grapefruit13
Copy link
Contributor

  1. 피그마 시안에는 top으로 올라가는 버튼이 있는데 이것도 넣는건가영
    image

@grapefruit13
Copy link
Contributor

  1. 기능소개 섹션의 고급통계분석 이미지가 좀 깨져보이는 것 같슴다. 이미지 화질 더 좋게 뽑으면 좋을 것 같은데 가능할까요?
    image

@grapefruit13
Copy link
Contributor

  1. 의견을 남겨주세요 섹션의 bg 컬러가 피그마에서는 회색인데, 현재 코드에서는 초록색인것 같아여. 디자인 변경된건가영

148 브랜치

image

피그마

image

@plla2
Copy link
Member Author

plla2 commented Mar 26, 2025

  1. 피그마 시안에는 top으로 올라가는 버튼이 있는데 이것도 넣는건가영
    image

해당 작업 pr 한번 올리고 추가 커밋으로 작업했습니다! 위의 커밋내역 제일 아래에서 확인가능합니다!

  1. 의견을 남겨주세요 섹션의 bg 컬러가 피그마에서는 회색인데, 현재 코드에서는 초록색인것 같아여. 디자인 변경된건가영

148 브랜치

image

피그마

image

이부분도 플로팅버튼과 마찬가지로 pr을 한번 올리고 추가 작업해서 커밋내역 하단에서 확인가능합니다!

  1. 기능소개 섹션의 고급통계분석 이미지가 좀 깨져보이는 것 같슴다. 이미지 화질 더 좋게 뽑으면 좋을 것 같은데 가능할까요?
    image

이부분은 위에 next/imge 인라인 스타일 부분 코멘트에서 답글 달아드렸는데, 이미지가 화질때문에 깨졌다기 보단 네트워크탭에서 확인해보시면 해당 원본 이미지들이 96, 75로 되어있습니다! 근데 디자인상으로는 94,105로 수정하느라 비율이 깨져서 뭉개져보입니다! 이 부분이랑 서비스소개 두번째 사진만 피그마상에서 이미지 원본 크기가 작게 되어있는 부분 유미님한테 말씀드리겠습니다!

@grapefruit13
Copy link
Contributor

1,2 : 제가 dev pull을 한 번 더 받았어야했는데 😅 실수했네요
3. 그렇군여 넵 알겠습니다~

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.

우선 approve 해두겠습니다 고생하셨어여

@plla2 plla2 merged commit 9192db1 into dev Mar 26, 2025
@plla2 plla2 deleted the THKV-148 branch March 26, 2025 03:33
plla2 added a commit to plla2/NNplanner-FE that referenced this pull request Mar 26, 2025
Feat[THKV-148]: 랜딩페이지 작업
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