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

fix: 반응형 페이지 breakpoint 수정 #177

Merged
merged 12 commits into from
Oct 4, 2023

Conversation

chaeyeon-yang
Copy link
Member

PR Type

  • 코드 스타일 변경 (Code style update) (formatting, local variables)

작업 내용

  • breakpoint 수정

Before

After

To Reviewers (참고 사항, 첨부 자료 등)

@chaeyeon-yang chaeyeon-yang added the 🎨 Style style, css, layout label Oct 1, 2023
@chaeyeon-yang chaeyeon-yang requested a review from a team October 1, 2023 14:13
@chaeyeon-yang chaeyeon-yang self-assigned this Oct 1, 2023
@chaeyeon-yang chaeyeon-yang changed the title fix: 반응형 페이지 breakpoint 수정 (이용안내 제외) fix: 반응형 페이지 breakpoint 수정 Oct 1, 2023
Copy link
Member

@heehminh heehminh left a comment

Choose a reason for hiding this comment

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

👍👍👍

@@ -67,7 +67,7 @@ const HeaderContents = ({ isLoading, userRes }: HeaderContentsProps) => {

return (
<>
<div className="flex justify-between items-center w-full my-8 relative lg:hidden">
<div className="flex justify-between items-center w-full my-8 relative smMaxLg:hidden">
Copy link
Contributor

Choose a reason for hiding this comment

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

음 smMaxLg 이 키워드를 smMaxLg: { max: "1024px", min: "360px" } 이렇게 tailwind에서 지정해놔서 소형 기기에서는 적용이 안 될 것 같아요. 1024px 아래부터 hidden을 적용하려면, 이전 코드가 더 맞을 것 같습니다.. ! 아래 처럼 헤더가 사라지지 않고 깨져서 나오네요.

image

@chaeyeon-yang

Copy link
Member Author

Choose a reason for hiding this comment

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

음 smMaxLg 이 키워드를 smMaxLg: { max: "1024px", min: "360px" } 이렇게 tailwind에서 지정해놔서 소형 기기에서는 적용이 안 될 것 같아요. 1024px 아래부터 hidden을 적용하려면, 이전 코드가 더 맞을 것 같습니다.. ! 아래 처럼 헤더가 사라지지 않고 깨져서 나오네요.

image @chaeyeon-yang

감사합니다 수정했습니다!!

@ShinChanU
Copy link
Contributor

머지하겠습니다 ~!

@ShinChanU ShinChanU merged commit 8cf039a into UPbrella:develop Oct 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🎨 Style style, css, layout
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants