Skip to content

Conversation

wzrabbit
Copy link
Collaborator

@wzrabbit wzrabbit commented Oct 19, 2023

[FE] 피드 채팅창 메뉴의 텍스트 오버플로우로 인한 높이가 변하는 문제를 해결

이슈번호

close #718

PR 내용

반드시 리뷰 또는 승인 전, "의논할 거리" 를 먼저 읽어 보십시오.

272471774-3d57118f-da47-45b6-ba49-c888634c7c49

본 PR에서는 피드 채팅창 메뉴에서 "공지로 등록" 글씨가 여러 줄로 넘쳐 피드의 높이가 의도치 않게 변하는 문제를 해결하였다.

  • 피드의 너비가 글씨를 표시하기 힘들 정도로 너무 좁은 경우, 글씨를 보이지 않게(display: none) 하여 피드의 너비가 의도치 않게 늘어나는 현상을 해결하였다.

참고자료

_2023_10_19_15_37_22_115.mp4

의논할 거리

문제 자체는 해결했지만, 두 가지의 문제가 있어 보여서 의견을 들어보고자 여기에 이에 대해 설명해 보고자 해
문제를 해결하기 위해 css의 @container (컨테이너 쿼리) 를 사용했는데, 이게 너무 최신 문법이어서 이렇게라도 해서 해결할 지 보류할 지 고민 중이야.

우선 컨테이너 쿼리를 설명하자면, 컨테이너 쿼리는 미디어 쿼리(@media) 와 매우 비슷하다고 생각하면 될 것 같아. 다만 미디어 쿼리는 브라우저의 가로/세로 길이에 따라 각 요소에 어떻게 속성을 적용할 수 있을지를 결정할 수 있는 쿼리라면, 컨테이너 쿼리는 특정 요소를 기준으로 어떻게 속성을 적용할 수 있을지를 결정할 수 있는 쿼리야. 꽤나 최신 문법이야.

이번 PR에서는 피드 페이지 컴포넌트의 크기를 기준으로 "공지로 등록" 글씨가 보이게 할지, 그렇지 않도록 할지를 @container 를 사용하여 적용한 거야.

image

아래의 두 글을 참고하면 좋을 것 같아!

  1. vscode에서 컨테이너 관련 속성들을 전혀 인식하지 못 하는 것으로 보여. 문법이 잘못된 것은 아닌데, vscode의 IntelliSense가 이 속성들을 아직 인식 못 하는 것으로 보여.
    image

이 PR에 따르면 @container 와 관련된 속성들을 v1.84 부터 지원할 예정이래, 그런데 아직은 v1.84 가 배포되지는 않은 상태야.

  1. 지원율이 아주 높지는 않은 것으로 보이는데, 이 caniuse 링크로 접속해서 같이 판단해 주면 좋을 것 같아

Copy link
Collaborator

@suyoungj suyoungj left a comment

Choose a reason for hiding this comment

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

👍 �이 정도 지원율이면 우리 타겟 브라우저용으로는 괜찮을듯??

@hafnium1923 hafnium1923 merged commit 4ddc5a3 into develop Oct 19, 2023
@hafnium1923 hafnium1923 deleted the fix/fe/피드-공지로등록-글씨-오버플로우로-인한-잘못된-레이아웃-고치기 branch October 19, 2023 08:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[FE] 피드 채팅창 메뉴의 텍스트 오버플로우로 인한 높이가 변하는 문제를 해결한다.
3 participants