SW13-Monstera / frontend

[소프트웨어 마에스트로 13기] CS Broker💥 ( service + admin 모노레포 )
https://csbroker.io
22 stars 1 forks source link

MSTR-329: 객관식 정답 미리 불러서 정답 한 개면 radio 버튼으로 수정 #60

Closed Kim-Hyunjo closed 2 years ago

Kim-Hyunjo commented 2 years ago

작업 내역

구현 내용 및 작업 했던 내역

작업 유형

netlify[bot] commented 2 years ago

Deploy Preview for elegant-yeot-fdb43b ready!

Name Link
Latest commit 5e8c678f31e7a2ff8f02b1d3ef60178191e78a77
Latest deploy log https://app.netlify.com/sites/elegant-yeot-fdb43b/deploys/6365007cb2618200086acbd7
Deploy Preview https://deploy-preview-60--elegant-yeot-fdb43b.netlify.app/
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

Kim-Hyunjo commented 2 years ago

앗 제가 방금 센터를 나왔는데요 집가서 보여드릴게요

라디오 버튼은 선택지가 하나일때만 쓰는데요, 기존의 체크박스와 다른점은 선택지 한개가 선택되어있을때 다른 선택지를 선택하면 기존것이 취소되고 새로 선택된다는 점입니다~!! 체크박스는 복수정답일때 사용되구요!

Kim-Hyunjo commented 2 years ago

코드는 문제 없는 것 같고.. 왜 라디오버튼이랑 체크박스로 나뉜건가요? 캡쳐같은거 보여주실 수 있나요?

객관식 정답 한개인 경우 (라디오버튼)

ezgif com-gif-maker (7)

객관식 정답 여러개인 경우 (체크박스)

ezgif com-gif-maker (8)

kshired commented 2 years ago

문제유형에 따라 선택 버튼의 디자인이 바뀌는건 이상한 것 같아요. 라디오 버튼에 디자인 입혀서 똑같이 사각형으로 못바꾸려나요?

Kim-Hyunjo commented 2 years ago

문제유형에 따라 선택 버튼의 디자인이 바뀌는건 이상한 것 같아요. 라디오 버튼에 디자인 입혀서 똑같이 사각형으로 못바꾸려나요?

원래 라디오버튼은 한개만 선택되고, 체크박스는 여러개를 선택할 수 있으니까 이 사실을 사람들이 명시적으로 인지할 수 있어 UX면에서 좋다고 생각했는데 그럴수도 있겠네요! 사실 이번 PR은 기능적인 것이고 실질적인 디자인 적용은 MSTR-337에서 진행할건데 두가지 방법이 있을 것 같아요.

  1. UI 디자인대로 라디오와 체크박스를 아예 없앤다.

    스크린샷 2022-11-05 오전 2 11 49
  2. 라디오/체크박스로 나눈다. (아래는 엘리스 예시)

    스크린샷 2022-11-05 오전 2 06 55 스크린샷 2022-11-05 오전 2 07 02

답안이 한개일때도 체크박스로 표시되는 것이 유저입장에서는 혼란을 야기할 수 있어 제 생각에는 체크박스로 통일하는 건 좋은 방향은 아닐 수 있을 것 같네요!

kshired commented 2 years ago

원래 라디오버튼은 한개만 선택되고, 체크박스는 여러개를 선택할 수 있으니까 이 사실을 사람들이 명시적으로 인지할 수 있어

위 사항을 대부분이 아는.. 사실인가요? 나만 몰랐어!!!

그건 그렇다치고, 통일성을 위해 저는 1번 하고 싶어요

Kim-Hyunjo commented 2 years ago

원래 라디오버튼은 한개만 선택되고, 체크박스는 여러개를 선택할 수 있으니까 이 사실을 사람들이 명시적으로 인지할 수 있어

위 사항을 대부분이 아는.. 사실인가요? 나만 몰랐어!!!

그건 그렇다치고, 통일성을 위해 저는 1번 하고 싶어요

시른데요?