시작페이지 뷰 생성 (이미지 이슈) #29
Conversation
JUDONGHYEOK
commented
Jul 5, 2021
JUDONGHYEOK
commented
- 시작페이지 구현
- 로그인, 회원가입 모달창
- 초기 화면 이미지 이슈 존재
| <Switch> | ||
| <Route path="/"> | ||
| <Route path="/home"> | ||
| <Home></Home> | ||
| </Route> | ||
| <NavBar path="/"></NavBar> | ||
| </Switch> |
There was a problem hiding this comment.
혹시 switch를 2개를 따로 두는 이유가 있을까요??
There was a problem hiding this comment.
NavBar는 한 개로 통일해서 Router 하단에 두되 각 페이지 라우팅 될 때마다 뷰 컴포넌트에 조건을 거는건 어떨까요?? 예를 들어 home에서는 Login 버튼을 띄워주고 로그인이 되고 메인페이지를 가면 Login버튼 컴포넌트를 비활성화하고 프로필 사진이랑 알람 버튼을 활성화 시키는 방식으로 하면 가능하지 않을까 싶습니다!
There was a problem hiding this comment.
첫 페이지를 home으로 routing네이밍을 하는 것도 좋지만 시작페이지이기도 하니 root로 설정해놓는게 어떨까요??
There was a problem hiding this comment.
NavBar의 경우 그렇게 해도 되지만 css가 차이가 있어서 맞추는데 협의나 시간이 걸릴 것 같아서 일단 그렇게 해놓았구요 switch를 두번 쓴거는 NavBar를 다르게 하기 위해서 했습니다. home에 걸려서 기존 NavBar가 무시되니까요 엔드포인트 네이밍은 제생각에는 제가 한 시작페이지를 /로 지정해놓고 재현님이 하신 메인페이지를 /projects와 같이 하는게 더 좋을것 같다는 생각입니다 후에 /projects에서 뻗어 나갈 때 /projects/create나 projects/:id 와 같이 하면 의미가 명확해질 것 같아서요
There was a problem hiding this comment.
안그래도 이번 commit 에서 routing name project로 수정했는데 생각이 같았네요ㅎㅎ 일단은 navbar 같은 경우는 다른거 다하고 리팩토링 하는걸로 해요~
