허들 툴바 구현

[handje]

Pull request

Related issue

Resolve #60 @handje

Motivation and context

• 허들 도구 모음을 위한 toggle group 구현
• 마이크, 비디오, 화면공유 버튼

Solution

Toggle

• 버튼 선택 여부에 따라 스타일 변경
• 외부선없는 default / 외부선있는 outline
• 기본 크기 default / 큰 화면용 lg

<Toggle
  size="default"
  variant="outline"
>
  {/* 토글 아이템*/}
</Toggle>

toggle-group

• ToggleGroup: 모양 default/ouline , 크기 default/lg , 선택속성: single/multiple 설정
• ToggleGroupItem: 고유한값인 value설정

    <ToggleGroup variant="outline" type="multiple">
      <ToggleGroupItem value="a">
          {/* 토글 아이템a */}
      </ToggleGroupItem>
      <ToggleGroupItem value="b">
          {/* 토글 아이템b */}
      </ToggleGroupItem>
      <ToggleGroupItem value="c">
         {/* 토글 아이템c */}
      </ToggleGroupItem>
    </ToggleGroup>

How has this been tested

Toggle

ToggleGroup

Slack레퍼런스

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)

Checklist

• My code follows the code style of this project.
• My change requires a change to the documentation.
• I have updated the documentation accordingly.
• I have read the docs/CONTRIBUTING.md document.
• I have added tests to cover my changes.
• All new and existing tests passed.

[KimKyuHoi]

생각보다 버튼이 엄청 작은데 오히려 버튼 Icon쪽을 차라리 쓰는게 낫지 않을까요? 제가 알기로 #15이슈에서 보류하기로 했던걸로 알고 있는데 구현하셨군요. 코드자체는 나쁘지 않은 것 같긴한데 한번 고려해보시면 좋을 것 같습니다.

[KimKyuHoi]

아이콘 버튼일때와 토글 group일때의 차이입니다.
정 아니면 토글 그룹을 조금 더 키워봐도 되지않을까 싶습니다.

[handje]

생각보다 버튼이 엄청 작은데 오히려 버튼 Icon쪽을 차라리 쓰는게 낫지 않을까요? 제가 알기로 #15이슈에서 보류하기로 했던걸로 알고 있는데 구현하셨군요. 코드자체는 나쁘지 않은 것 같긴한데 한번 고려해보시면 좋을 것 같습니다.

#15는 댓글 관련 토글그룹이라고만 알고있고 상세 내용을 숙지하지 못했습니다!
말씀하신 크기차이는 토글그룹의 size='lg' 일 경우 , button의 icon속성과 동일합니다!

폴더구조만 Tooggle로 합친 뒤, conflict해결 후 다시 리뷰 요청드렸습니다!

[KimKyuHoi]

생각보다 버튼이 엄청 작은데 오히려 버튼 Icon쪽을 차라리 쓰는게 낫지 않을까요? 제가 알기로 #15이슈에서 보류하기로 했던걸로 알고 있는데 구현하셨군요. 코드자체는 나쁘지 않은 것 같긴한데 한번 고려해보시면 좋을 것 같습니다.

#15는 댓글 관련 토글그룹이라고만 알고있고 상세 내용을 숙지하지 못했습니다! 말씀하신 크기차이는 토글그룹의 size='lg' 일 경우 , button의 icon속성과 동일합니다!

폴더구조만 Tooggle로 합친 뒤, conflict해결 후 다시 리뷰 요청드렸습니다!

그럼 그대로 가시죠! 제가 이해를 잘 못했던 것 같네요!

[KimKyuHoi]

수정만 가시고 그대로 갑시다!