-
Notifications
You must be signed in to change notification settings - Fork 0
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
[FE] feat: 프론트엔드 home 기능 구현 #33
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
tailwindcss 잘 활용해보세요~ 편리할듯.
frontEnd/src/components/Button.tsx
Outdated
@@ -0,0 +1,14 @@ | |||
import { ReactNode } from 'react'; | |||
|
|||
export default function Button({ children, onClick, fontSize }: { children: ReactNode; onClick: () => void; fontSize: string }) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
타입정의가 길면 type이나 interface로 선언~
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
type ButtonProps = {
children: ReactNode;
onClick: () => void;
fontSize: string;
};
선언해주었습니다!
frontEnd/src/hooks/useInput.ts
Outdated
const [value, setValue] = useState(initialValue); | ||
|
||
const onChange = (event: React.ChangeEvent<HTMLInputElement>) => { | ||
if (event.target) setValue(event.target.value); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
해체할당문법으로 받기.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
const {
target: { value },
} = event;
요런 형태로 개선해보았습니다!
15d1df2
to
321dd29
Compare
mainColor,backgroundColor,black,white 설정
main이미지 추가 html, body width, height설정
children, onClick, fontSize를 props로 받는 공용 컴포넌트인 Button컴포넌트 추가
Home화면 UI구성 (#32)
uuid 라이브러리 설치 uuid types설치 (#32)
방생성 버튼을 누르는 경우, UUID값에 따라 방을 새롭게 생성하고, 해당 방으로 이동하는 기능 구현 (#32)
input의 state, change함수를 추상화한 useInput 훅 추가 (#32)
방참가 입력값이 있는경우 참가 버튼에 불이 들어오는 기능 구현 input창에 입력한 방으로 이동하는 기능 구현 input창에 입력한 후 클릭하지 않고 enter를 눌러도 이동하는 기능 구현 (#32)
tailwind 반응형 기준 추가 (#32)
Home 로고 이미지 변경
모바일 반응형 대응 (#32)
useInput event 구조분해할당으로 코드 개선 (#32)
Button 공통 컴포넌트 props타입 명시 (#32)
input placeholder 추가
방생성 클릭시, uuid를 url로 갖는 새로운 방을 생성하고, 해당 방 url로 이동한다. 참가 클릭시, input입력창에 입력한 url을 가진 방으로 이동한다. (#32)
1518f10
to
a491488
Compare
불필요한 async await 구문삭제
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
100vh -> h-screen 100vw -> w-screen Co-authored-by: Wiyeong Seo <[email protected]>
PR 설명
프론트엔드 home 기능 구현
✅ 완료한 기능 명세
📸 스크린샷
기본 접속시
방번호 입력시
방생성 버튼 클릭시
고민과 해결과정
useInput
input로직이 나중에도 필요할 수 있을 텐데 해당 로직에 필요한 훅을 모아서 useInput으로 만들어 주었다.
uuid
방을 어떻게 만들어 줄까하다가 uuid 라이브러리를 사용해서 처리해주었다