fix(Select): border 유무에 따라 size가 변형되지 않도록 수정합니다. #183
+6
−1
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
변경사항
close #182
#174 에 설명되어있듯이,
triggerContent
에 대한focus
style을 별도로 정의하게 되면서S.input
스타일을 가져다가 사용하지 않게 되었고,S.input
에 정의되어 있는border: 1px solid transparent
가 적용되지 않아서 border 유무에 따라 size가 변형되어 다른 요소가 밀리는 이슈가 발생했어요.이를 해결하기 위해 별도로 정의한
focus
style에 정의되어 있던 border를 box-shadow inset으로 변경하여 테두리가 생기더라도 요소의 size가 변형되지 않도록 하여 문제를 해결했어요.이렇게 되면 Select에서 Input의 공통 style을 참조하고 있지 않는데, Select를 Input으로 묶기보다 별도로 분리하는 건 어떤지에 대한 의견을 들어보고 싶어요!!
링크
https://sopt-makers.slack.com/lists/T040QGZF77H/F07LCJ586TS?record_id=Rec07T75VU4T1
시급한 정도
🏃♂️ 보통 : 최대한 빠르게 리뷰 부탁드립니다.