[Park] Avatar Size 지정 방법 추가 #46
Conversation
✅ Deploy Preview for idyllic-puffpuff-476e23 ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
There was a problem hiding this comment.
Custom 하게 사용되는 사이즈가 동일한 사이즈로 (혹은 비슷한 사이즈로) 많은 곳에서 쓰이게 되면 그 때 지정된 사이즈를 추가 하는 방식이 맞다고 생각이 들었습니다.
User Detail 에 들어가는 Avatar 사이즈는 단 한 곳에서만 쓰이기 때문에 xxxLarge 등의 키워드로 추가하는 것이 되려 불편
확실히 한 곳에서만 사용되는 크기 값 때문에 속성이 추가 되는건 앞으로도 크기의 변경이 있을 때마다 값이 늘어나는 변경사항이 발생하는 느낌이라 편하지 않을 것 같긴하네요.
또한 저도 개발을 할 때 중복되는 값은 중복이 발생 할 때 분리를 하는게 맞다고 생각을 해서 말씀해주신 의견에는 공감이 가는 것 같아요.
하지만 그렇다고 너무 자유롭게 커스텀 할 수 있게 만드는건 시각적으로 일관된 사용자 경험을 제공하는 디자인 시스템의 기본 목적에서 살짝 벗어날 수 있는게 아닌가 싶은 우려가 드네요.
그래서 사이즈를 어느정도 커스텀 할 수 있게 한다고 해도 제한은 조금 두는게 좋지 않을까 싶은 생각이 들어요.
px이나 rem 단위의 제한이라던가, 늘릴 수 있는 최대 사이즈? 또 아니면 Spacing 처럼 특정 비율로 증가되게 한다던가?!
코드는 LGTM 입니당 ^_^)b
| export type SizeSX = { | ||
| width?: string; | ||
| maxWidth?: string; | ||
| minWidth?: string; | ||
| height?: string; | ||
| maxHeight?: string; | ||
| minHeight?: string; | ||
| }; |
There was a problem hiding this comment.
sizeSX를 따로 분리한건 유용할 것 같네요! ^0^)b
이 발표가 갑자기 떠올랐는데, 비슷한 사례를 겪으셨던 것 같아요.
그래서 linear 에서는 아래와 같이 지향하신다고 하셨는데
저희가 디자이너가 없다보니 "이상하다" 라고 느낄 수 있는 사항이 무엇인지 지금은 모르겠지만(어쩌면 햄디가 말씀하신 단위의 느낌일수도 있고?) 디자이너의 표현의 제약이 생긴다는 부분은 공감이 되는 부분 같아요. 그래서 전 이 방법이 현 시점에서는 최선이지 않을까? 싶긴 한데, 다른 의견 더 있으실지 궁금하네요 |
|
참고 자료 감사합니다! ^_^)b 제가 너무 자유로운게 아닌가 싶다고 느꼈던 포인트는 저희가 SpacingSX를 추가하면서 저희 디자인 시스템에서 margin, padding 값(여백이나 규격) 또한 제한을 두는 요소라고 정했어서 그렇다면 커스텀 할 수 있는 크기에도 어느 정도 제한을 둬야 하지 않을까 싶었습니다!
하지만 말씀해주신 위 의견에도 공감이 가고 해당 방법이 현 시점의 최선이라는 의견에 완전 동의합니다^_^)b |
close #45
디자인 상 지정된 사이즈 이외에도 다양한 사이즈가 필요할 수밖에 없다고 생각이 들어 SizeSX 를 활용해 지정하는 방법을 추가했습니다.
디자인 상 지정된 사이즈를 사용하는 것은 편리함과 동시에 제한을 두어 강제하는 느낌도 있는데,
Custom 하게 사용되는 사이즈가 동일한 사이즈로 (혹은 비슷한 사이즈로) 많은 곳에서 쓰이게 되면 그 때 지정된 사이즈를 추가 하는 방식이 맞다고 생각이 들었습니다.
이에 대해 어떻게 생각하시는지도 궁금합니다.
(일단 해당 SX 를 추가한 이유는 User Detail 에 들어가는 Avatar 사이즈는 단 한 곳에서만 쓰이기 때문에 xxxLarge 등의 키워드로 추가하는 것이 되려 불편해보였습니다. xxxLarge 가 어디서 쓰이는 지 추적도 어렵고..)
SIzeSX 는 FlexBox 컴포넌트에 있길래 styles/size.ts 로 별도 분리하였습니다.