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

[Feature] - 랜딩 페이지 구현 #539

Merged
merged 19 commits into from
Oct 23, 2024
Merged

[Feature] - 랜딩 페이지 구현 #539

merged 19 commits into from
Oct 23, 2024

Conversation

0jenn0
Copy link

@0jenn0 0jenn0 commented Oct 19, 2024

✅ 작업 내용

  • 랜딩 페이지 구현

📸 스크린샷

2024-10-19.4.38.40.mp4

🙈 참고 사항

Copy link

github-actions bot commented Oct 19, 2024

Test Results

11 tests   11 ✅  14s ⏱️
 2 suites   0 💤
 1 files     0 ❌

Results for commit ae1dde5.

♻️ This comment has been updated with latest results.

@0jenn0 0jenn0 self-assigned this Oct 20, 2024
@simorimi simorimi added the FE label Oct 21, 2024
@simorimi simorimi added this to the sprint 6 milestone Oct 21, 2024
Copy link
Contributor

@jinyoung234 jinyoung234 left a comment

Choose a reason for hiding this comment

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

image

띄워쓰기(여행을 따라가고, 여행을 어떻게)

image
텍스트 개행 + 깨지는 부분

이 부분들만 수정되면 좋을거 같아요!!

랜딩 페이지 구현 정말 빡셌을거 같은데 구현 잘 해주셔서 감사합니다 :) 작업 고생하셨어요!!

(entries) => {
const updates: { [key: string]: AnimationState } = {};
entries.forEach((entry) => {
console.log("entry", entry);
Copy link
Contributor

Choose a reason for hiding this comment

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

console.log들이 보여요!

Copy link
Author

Choose a reason for hiding this comment

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

오 이런 바로 삭제하겠습니다

@0jenn0
Copy link
Author

0jenn0 commented Oct 21, 2024

스크린샷 2024-10-21 오후 7 34 29 스크린샷 2024-10-21 오후 7 34 19

깨지던 부분 수정했습니다!

Copy link

@simorimi simorimi left a comment

Choose a reason for hiding this comment

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

리버 랜딩페이지 작업량을 보니 어마어마 하네요
이번 스프린트 고생 많으셨습니다
리뷰 몇 개 남겨봤는데 참고 바랄게요!

@@ -1 +1,7 @@
export { default as ExcitedTturi } from "./excitedTturi.webp";
export { default as BigTturi } from "./bigTturi.webp";

Choose a reason for hiding this comment

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

빅뚜리

${({ theme, $textType = "body" }) => theme.typography.mobile[$textType]}
${({ $isInline }) => ($isInline === true ? "display: inline" : "")}

Choose a reason for hiding this comment

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

inline 여부에 따라 inline 과 block 식으로 해도 괜찮을 거 같은데 이 부분은 참고만 해요!

Copy link
Author

Choose a reason for hiding this comment

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

그게 더 깔끔하겠네요 수정햇습니다

</div>
</S.Container>
<S.Gradient />
<S.Image src={BigTturi} alt="" />

Choose a reason for hiding this comment

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

alt 값을 일부러 비우신 거 같은데 의도하신 바가 무엇인가요?

Copy link
Author

Choose a reason for hiding this comment

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

단순히 장식용 이미지라 빈 alt값을 주었습니다!

@@ -1,6 +1,7 @@
export const ROUTE_PATHS_MAP = {
back: -1,
root: "/",
main: "main",

Choose a reason for hiding this comment

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

해당 부분 메인으로 옮김에 따라 홈버튼 부분의 경우 main으로 라우팅 수정해야할 거 같다는 생각이 듭니다!

Copy link
Author

Choose a reason for hiding this comment

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

오 그렇네요 짚어주셔서 감사합니다!

@0jenn0 0jenn0 merged commit 8cab7f8 into develop/fe Oct 23, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants