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

[Feat] 홈, 회원가입, 로그인 페이지 작성 #57

Merged
merged 9 commits into from
Nov 11, 2024
Merged

[Feat] 홈, 회원가입, 로그인 페이지 작성 #57

merged 9 commits into from
Nov 11, 2024

Conversation

PMtHk
Copy link
Collaborator

@PMtHk PMtHk commented Nov 11, 2024

관련 이슈 번호

close #51

#25 #22

작업 내용

  • 랜딩페이지 작성했습니다.
  • 로그인 및 회원가입 페이지 작성했습니다.

고민 사항

React Hook Form 과 zod 를 도입해야 하나 고민해보았습니다.

아직 프로젝트 사이즈가 큰 것도 아니고, 관리해야 할 Form 이 많지 않아
커스텀 훅으로 일단 작성해보려고 합니다.
이에 대한 내용은 여기에 있습니다.

스크린샷

스크린샷 2024-11-11 오후 3 08 59 스크린샷 2024-11-11 오후 7 25 34 스크린샷 2024-11-11 오후 7 25 32

@PMtHk PMtHk added Feature 새로운 기능을 추가 FE 클라이언트 작업 labels Nov 11, 2024
@PMtHk PMtHk self-assigned this Nov 11, 2024
Copy link
Collaborator

@yangchef1 yangchef1 left a comment

Choose a reason for hiding this comment

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

랜딩 페이지 너무 예뻐요 ,,, 🤩
집에 디자이너 숨겨놓으셨죠 ,,,

@jjeonghak
Copy link
Collaborator

넘나 잘만드셨네용

@PMtHk PMtHk merged commit bbc349b into dev Nov 11, 2024
2 checks passed
@PMtHk PMtHk deleted the feat/#51 branch November 11, 2024 12:21
Copy link
Collaborator

@iam454 iam454 left a comment

Choose a reason for hiding this comment

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

로고 작업을 해놓으셨구나 훔쳐써야겠습니다.
근데 fill 색상을 반대로 해놓으셨는데 의도하신건가용?!

추가로, tanstack router 사용하니까
pages 폴더를 만들지 않고, routes 폴더로만 관리하는 건 어떤가요?
추후에 tanstack query 도입하게 되면 한 파일에 있어야할 이유가 생길 수 있을 것 같아요.

@@ -24,5 +24,6 @@ module.exports = {
'import/prefer-default-export': 0,
'react/require-default-props': 0,
'react/jsx-props-no-spreading': 0,
'react/prop-types': 0,
Copy link
Collaborator

Choose a reason for hiding this comment

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

주엽님도 이거 없앴군요.. ㅋㅋㅋㅋ

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

넵 ㅜㅜ

Comment on lines +16 to +26
useEffect(() => {
const handleScroll = () => {
const currentScrollY = window.scrollY;
setIsTop(currentScrollY <= 10);
};

handleScroll();
window.addEventListener('scroll', handleScroll, { passive: true });

return () => window.removeEventListener('scroll', handleScroll);
}, []);
Copy link
Collaborator

Choose a reason for hiding this comment

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

이거 어떤 역할을 하나요??

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

스크롤 위치에 따라서 Header 의 아래에 border 및 shadow 를 보여줄 지 정하기 위해 사용됩니다.
현재 로그인/회원가입 페이지에서는 확인하실 수 있습니다~

Comment on lines +44 to +70
<header className={headerClasses}>
{/* Logo Section */}
<div className="flex items-center space-x-4">
<Link to="/" className="transition-opacity hover:opacity-80">
<Harmony size={32} />
</Link>
</div>

{/* Navigation Buttons */}
<nav className="flex items-center space-x-3">
{!isLoginPage && (
<Button variant={isSignupPage ? 'outline' : 'ghost'} asChild>
<Link to="/login" className="hover:text-primary font-medium transition-colors">
로그인
</Link>
</Button>
)}

{!isSignupPage && (
<Button variant="outline" asChild>
<Link to="/signup" className="hover:text-primary font-medium transition-colors">
회원가입
</Link>
</Button>
)}
</nav>
</header>
Copy link
Collaborator

Choose a reason for hiding this comment

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

조건부 렌더링으로 처리하기보다
사용하는 쪽에서 주입하는 쪽이 더 좋을 것 같아요!

asChild는 덕분에 알았습니다. 👍

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

사용하는 쪽에서 path 를 주입하신다는 말씀이실까요?

Copy link
Collaborator

Choose a reason for hiding this comment

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

음.. 글로 쓰기가 어렵네요 시간이 늦어서 머리가 굳었나
낼 말로 이야기할게요 ㅋㅋㅋㅋ

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

좋습니다!

@@ -16,7 +42,7 @@ export default {
700: 'hsl(var(--green-700))',
},

white: 'var(--white)',
white: 'hsl(var(--white))',
Copy link
Collaborator

Choose a reason for hiding this comment

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

ㅋㅋㅋㅋ저도 이거 수정했었습니다..

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

저번에 오프라인 때 말씀드리고 수정버전을 안올려서 ㅎㅎ

@PMtHk
Copy link
Collaborator Author

PMtHk commented Nov 11, 2024

근데 fill 색상을 반대로 해놓으셨는데 의도하신건가용?!

svg 내부 요소들을 확인해보니, 라이트 모드 기준으로 path 태그들의 색상이 검정이어야 맞는 것 같아서
작업해뒀습니다.

추가로, tanstack router 사용하니까
pages 폴더를 만들지 않고, routes 폴더로만 관리하는 건 어떤가요?
추후에 tanstack query 도입하게 되면 한 파일에 있어야할 이유가 생길 수 있을 것 같아요.

routes 폴더 내 signup/login 등은 createFileRoute 를 하고 있어서
해당 파일에서 직접 구현체를 작성하기가 애매하다 판단하고 pages 를 분리했습니다.
이 부분은 이번 주 하루 정도 페어로 리팩토링 해봐도 좋을 것 같네요

@iam454
Copy link
Collaborator

iam454 commented Nov 11, 2024

근데 fill 색상을 반대로 해놓으셨는데 의도하신건가용?!
svg 내부 요소들을 확인해보니, 라이트 모드 기준으로 path 태그들의 색상이 검정이어야 맞는 것 같아서 작업해뒀습니다.

아하 이거 제가 질문의 의도를 잘 전달하지 못한 것 같습니다. ㅎㅎ;
요랬던게
Screenshot 2024-11-12 at 12 26 43 AM
요렇게 되어서
Screenshot 2024-11-12 at 12 27 01 AM
바꾸신 게 더 마음에 드신건가?! 이런 느낌이었습니다 ㅋㅋㅋ

@PMtHk
Copy link
Collaborator Author

PMtHk commented Nov 11, 2024

아하! 그렇군요 뇌이징이 되어서 바뀐지도 몰랐습니다 ㅜㅜ
수정하겠습니다...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
FE 클라이언트 작업 Feature 새로운 기능을 추가
Projects
None yet
Development

Successfully merging this pull request may close these issues.

랜딩 페이지를 작성한다.
4 participants