aicers / frontary

Reusable HTML components using Yew for AICE
Apache License 2.0
0 stars 0 forks source link

Add clumit component(radio, checkbox, tab menu, modal) #98

Closed dayeon5470 closed 3 months ago

dayeon5470 commented 3 months ago

Fixes #97

jinjoolee07 commented 3 months ago

Modal

Screenshot 2024-06-20 at 8 57 55 AM

Tab Menu

Screenshot 2024-06-20 at 9 04 35 AM

Radio

Screenshot 2024-06-20 at 9 04 38 AM

Checkbox Screenshot 2024-06-20 at 9 11 23 AM

지금 현재 frontary에 pumpkin-dark, pumpkin-light feature를 추가했습니다. 그리고 frontary에 clumit버전 css를 새로 만들어서 적용한 상태라 위에 보이는 사진처럼 modal, tab menu, radio, checkbox 부분을 제외하고는 깨져서 보이는 상태입니다. 나머지 부분은 다른 PR로 올릴 예정입니다.

sehkone commented 3 months ago

@div-seungha

PR 리뷰 시 해당 PR과 직접 관련이 없는 사항들은 별도 issue나 discussion에 등록해 주세요.

위에서 @div-seungha가 제기한 사항들은 좋은 의견들입니다. 다만, 이번 PR 이전에 이미 작성된 코드에 관련된 것들이고 더불어 이 부분에만 국한되는 사항이 아니기까지 하므로, 별도로 다루는 것이 바람직합니다.

또한, 평소에 생각한 의견은 언제라도 issue나 discussion에 적어 주세요.

예전부터 궁금했던 부분이었는데 아이콘을 svg가 아닌 png로 따서 쓰는 데에는 어떤 특별한 이유가 있는 것일까요? (궁금해서 여쭙는 질문입니다)

위와 같은 궁금한 점은 그때 그때 discussion에 적으면 좋습니다. 누군가를 특정해서 물어봐도 되고, 불특정하여 질문해도 좋습니다.

라디오 버튼은 여러가지 아이템을 나열하고 그 중에 단 한 개의 아이템을 선택하는 UI로 보통 라디오 그룹 안에 라디오 개별 버튼을 구현하는 방식으로, 웬만하면 <div>보다는 <input type="radio"><fieldset>, <label>, <legend>와 같은 태그들을 함께 활용해서 화면을 구현하는 식으로 많이 쓰입니다. <div>로 구현하다고 하더라도 role="radio" 속성을 넣어 주는 것이 좀 더 HTML을 의미론적으로 명확하게 사용하는 방법입니다.

아무런 내용이 없는 <div>를 보통은 쓰지 않습니다.

위 둘과 같은 사항들은 issue에 적으면 적절합니다.

물론, PR에 직접 관련된 것이 아니라도 심각한 수정 사항인 경우에는 PR에 적어서 반영되도록 하는 것이 더 낫고요.

질의나 의견 등 어떤 내용이라도 언제든지 제한없이 issue나 discussion에 적어 주세요. Issue 또는 discussion 작성 시 형식은 자유롭게 작성하면 되고, 템플릿을 참고해도 좋습니다.

sehkone commented 3 months ago

@sophie-cluml @div-seungha 별도로 적기로 한 두 가지 사항을 이슈로 등록해 주세요.