-
Notifications
You must be signed in to change notification settings - Fork 3
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
[#23] apply Detail page #25
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.
👍 👍 👍 👍 👍 👍 이렇게 변경 상황이 많은 경우에
브랜치에서 브랜치로 나눠서 PR은 보내는 식으로 PR 양을 줄일 수 있을거 가타요 ~~~~ 🥇
`; | ||
|
||
const ulWrapper = css` | ||
all: unset; |
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.
이런 속성을 글로벌하게 벗겨내는게 좋을거가타요~~
width: 100%; | ||
overflow-x: scroll; | ||
|
||
::-webkit-scrollbar { |
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.
::-webkit-scrollbar { | |
&::-webkit-scrollbar { |
요런 형식으로 되어야 적용되지 않을까요?!
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.
허걱 맞아요 !!!
</div> | ||
<div css={imgWrapper}> | ||
<Image | ||
src="/images/menu.jpg" |
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.
이미지의 public path의 경우 상수로써 관리하는게 어떨까 생각해요~!
설명
Detail page를 개발하였습니다.
각 식당마다 정보가 잘못되거나 수정할 부분이 있을 수 있어 ChangeInform page를 만들어 제보할 수 있도록 하였습니다.
지도, 공유 버튼은 개발 후 적용할 계획입니당
중점적으로 봐줬으면 좋을 부분