-
Notifications
You must be signed in to change notification settings - Fork 461
Open
Labels
area:uiGeneral user interface and experience improvementsGeneral user interface and experience improvementsarea:vue-migration
Description
Context
The MediaAssetViewModeToggle component in src/platform/assets/components/MediaAssetViewModeToggle.vue was added in PR #7729. During review, it was noted that:
- The component overrides many button styles
- It's a single-use component, which we're trying to move away from
Proposed Solution
Refactor this component to use the toggle-group component being introduced in PR #7767, which is based on shadcn-vue's toggle-group.
References
- Original PR: [QPOv2] Add media assets viewmode toggle #7729
- Comment thread: [QPOv2] Add media assets viewmode toggle #7729 (comment)
- Toggle-group PR: feat: add label support to boolean toggle widgets #7767
- Requested by: @benceruleanlu
┆Issue is synchronized with this Notion page by Unito
Metadata
Metadata
Assignees
Labels
area:uiGeneral user interface and experience improvementsGeneral user interface and experience improvementsarea:vue-migration