-
Notifications
You must be signed in to change notification settings - Fork 4.6k
Closed
Labels
Needs Design FeedbackNeeds general design feedback.Needs general design feedback.[Feature] Font Library[Feature] TypographyFont and typography-related issues and PRsFont and typography-related issues and PRs[Focus] Accessibility (a11y)Changes that impact accessibility and need corresponding review (e.g. markup changes).Changes that impact accessibility and need corresponding review (e.g. markup changes).[Status] In ProgressTracking issues with work in progressTracking issues with work in progress[Type] RegressionRelated to a regression in the latest releaseRelated to a regression in the latest release
Description
The "Manage fonts" butto (added in this pr) is far too prominent in this iteration, especially accounting for the entire panel's contents. It's adding unnecessary complexity to the UI, introducing another type of control to the already disjointed view.
I'm fine with testing alternatives, but not a solution that fundamentally exists outside of the editor design language. A list of choices, with a button all the way at the bottom is not ideal.
Commented originally here.
Metadata
Metadata
Assignees
Labels
Needs Design FeedbackNeeds general design feedback.Needs general design feedback.[Feature] Font Library[Feature] TypographyFont and typography-related issues and PRsFont and typography-related issues and PRs[Focus] Accessibility (a11y)Changes that impact accessibility and need corresponding review (e.g. markup changes).Changes that impact accessibility and need corresponding review (e.g. markup changes).[Status] In ProgressTracking issues with work in progressTracking issues with work in progress[Type] RegressionRelated to a regression in the latest releaseRelated to a regression in the latest release