search

the-kingdoms / modules-fe

1 stars 4 forks source link

FIX: MenuItem, InputMenu story 분리 및 InputMenu 구조 수정 #66

Closed Wonchang0314 closed 1 month ago

Wonchang0314 commented 1 month ago

inputMenu에서 props로 menuitem배열을 받지않고 나중에 사용할때 map를 돌려서 menuItem 컴포넌트를 자식요소로 받는 쪽으로 수정했습니다.

MenuItem이랑 InputMenu도 Story를 분리해서 따로 확인해볼 수 있게끔 수정했습니다

그리고 지금 MenuItem L사이즈, complex일때 높이가 피그마에 주어진 max-h값이상으로 가면 scroll되게끔 해놨는데 디자인 시안처럼 깔끔하게 떨어지지는 않는 것 같은데 어떻게 생각하시나요?

스크린샷 2024-08-01 오전 12 00 37
PangKiYeon commented 1 month ago
스크린샷 2024-08-01 오후 2 18 53

state 에 따른 icon 색깔 변화가 필요합니다. icon-left 상태의 글자와 icon의 gap이 디자인과 달라 수정이 필요해 보입니다.

PangKiYeon commented 1 month ago

menu item state 에서 hover 상태를 따로 뽑지 않고 enabled state 에서 hover 기능을 포함하는 것이 좋을 것 같습니다.

PangKiYeon commented 1 month ago

깔끔하게 안떨어지는 부분은 토요일에 디자이너분과 상의해보겠습니다!

Wonchang0314 commented 1 month ago

icon-left 상태의 글자와 icon의 gap이 디자인과 달라 수정이 필요해 보입니다 -> 해결 깔끔하게 안떨어지는 부분은 토요일에 디자이너분과 상의해보겠습니다! -> 내일 뵙시다 state 에 따른 icon 색깔 변화가 필요합니다. -> 해결