Skip to content

137 post page post header 플렉스 변경1#159

Open
seongseonko wants to merge 3 commits into
Finale-고성선from
137-post-page-post-header-플렉스-변경1

Hidden character warning

The head ref may contain hidden characters: "137-post-page-post-header-\ud50c\ub809\uc2a4-\ubcc0\uacbd1"
Open

137 post page post header 플렉스 변경1#159
seongseonko wants to merge 3 commits into
Finale-고성선from
137-post-page-post-header-플렉스-변경1

Conversation

@seongseonko

Copy link
Copy Markdown
Contributor

이 PR은 어떤 유형인가요?

  • 리팩터링
  • 기능
  • 버그 수정
  • 최적화
  • 문서 업데이트

설명

post page post header의 tablet, mobile 미디어 쿼리 적용 하였습니다.

관련 티켓 및 문서

스크린샷, 녹화

screend.mp4

변경사항을 테스트하는 방법, 테스트에 사용된 기기 및 브라우저, UI 변경에 대한 관련 이미지 등에 대한 지침을 이곳에 기재해 주세요.

UI 접근성 체크리스트

UI 변경 사항이 있는 경우, 이 체크리스트를 활용하세요:

  • Semantic HTML 구현?
  • 키보드 조작이 지원?
  • axe DevTools를 사용하여 Critical 및 Serious 문제를 확인하고 해결했나요?

[선택사항] 이 PR을 가장 잘 설명하는 GIF는 무엇인가요?

@seongseonko seongseonko added the enhancement New feature or request label Dec 23, 2023
@seongseonko seongseonko self-assigned this Dec 23, 2023

@nakyoung98 nakyoung98 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

제가 우려한 부분 한번만 체크해주세요!

`

function CountPerson({ result, direction = false }) {
${mediaQuery.tablet} {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

오 이렇게 안보이게 하신거군요!

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

그런데 혹시 이 경우에 카드리스트 페이지에서는 tablet 모드로 되면 안사라지나요?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

넵 카드리스트랑 달라서 괜찮아요!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants