the-kingdoms / modules-fe

1 stars 4 forks source link

FIX: 얼루가 컴포넌트 수정 #69

Closed Wonchang0314 closed 1 month ago

Wonchang0314 commented 1 month ago

일단 이슈 나온것들중에 버튼 위주로 수정해놨습니다

1번 - 버튼 관련 이슈 2번- Spacing 관련 이슈

1-1) 버튼 story 두개 따로 파기 -> 해결

1-2) 버튼 클릭, 호버시 border 생기는 만큼 사이즈가 늘어나는 문제 -> 해결

1-3) 버튼 focus는 누른 이후로 적용되는 스타일 active는 누르고 있을때만 적용되는 스타일 hover도 state를 따로 두는게 아니라 코드상에만 반영되게끔 -> 해결

1-4) 버튼 아이콘도 props로 받을 수 있게끔 만들기 -> 해결

2) Spacing story -> 이런식으로 확인만 가능하도록 수정했습니다.

스크린샷 2024-08-05 오후 3 51 05
PangKiYeon commented 1 month ago
스크린샷 2024-08-07 오후 5 10 05

button 색상 elevated_primary , 클릭했을 때 border가 존재하지 않습니다

PangKiYeon commented 1 month ago

button 아이콘 키가 드롭다운으로 설정이 안되어있습니다. 선아님 답변 참고해 수정해주세요! ( https://github.com/the-kingdoms/modules-fe/issues/70 ) 그리고 button 에서 iconkey가 필수 props 일 필요는 없을 것 같습니다!

스크린샷 2024-08-07 오후 5 18 44
PangKiYeon commented 1 month ago

large 일 때 icon 크기랑 medium 일 때 아이콘 크기가 같아서 피그마에 맞게 수정이 필요할 것 같습니다. 그리고 이러한 오류가 발생하니 수정 부탁드립니다.

스크린샷 2024-08-07 오후 5 26 23
PangKiYeon commented 1 month ago

button state 에 active 상태가 아직 존재해서 이 부분 수정 부탁드립니다.

스크린샷 2024-08-07 오후 5 31 27
Wonchang0314 commented 1 month ago

button state 에 active 상태가 아직 존재 => 이 부분은 hover랑 focus빼고 active는 남기는걸로 기억했는데 아니였나요?

PangKiYeon commented 1 month ago

button state 에 active 상태가 아직 존재 => 이 부분은 hover랑 focus빼고 active는 남기는걸로 기억했는데 아니였나요?

status = active 를 선택할 경우가 없기에 삭제하기로 했습니다 ( https://github.com/the-kingdoms/modules-fe/issues/70 )