-
Notifications
You must be signed in to change notification settings - Fork 4.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Font Library: consider making the library more discoverable #55179
Comments
As a conversation has continued over in both this PR to replace the Aa icon with text and an issue to replace the Aa icon with text, I wanted to bring in a current design proposal to discuss and move forward for WordPress 6.5:
![]() The feedback about the above proposed design highlights how Aa icon remains unclear to leave in place after fonts are installed. One can imagine a situation where they click Typography > Add Fonts > they add a font > and then they are unable to understand where to go next to manage those fonts. While it's not necessarily something of primary importance, discoverability and that initial drop off are (still) coming up as points of feedback. @t-hamano suggested adding using the edit pencil button that appears throughout the site editor interface to communicate the ability to edit something a next step. What iterations can be done as feedback and various solutions are being explored? cc @afercia @jasmussen @richtabor @t-hamano @carolinan @earnjam |
The following are showing various icons that can be used instead of a new icon, which seems to be part of what's causing confusion. After someone installed fonts, this icon is what would then be used to manage them. This feels like it strikes a good balance of familiarity throughout the interface and matches the level of importance (installing fonts primary vs managing fonts lower). The key is figuring out which works best. Edit Pencil icon edit.icon.movThis matches what @t-hamano suggested before! + button icon Another option that @richtabor brought up as we were chatting about this is the + button, which matches what's in place for colors: plus.button.movSettings icon @richtabor also brought up the settings icon which might better reflect the "manage" aspect: settings.icon.movSummary Finally, wanted to note this issue around discoverability by adding a command to the command palette to open up fonts as well which should also help #54880 |
I like the settings icon when there are fonts; + icon when there are not. It's already familiar and expected. |
My feelings about each icon are as follows, and I prefer the
Based on this, I would like to propose the following new experience.
435daef84f63e75b64c1ccf031b96a68.mp4Alternatively, we might want to display the text "No fonts installed" and a plus icon if no fonts are available. |
I'm not sure changing icon conditionally would contribute to UI clarity and help users.
I would have preferred to not use button icons at all. Ideally, these kind of alternative design choices should just be A-B tested with a group of testers of diverse abilities instead of being left to personal argumentation. In absence of user testing, there's prior research and guidelines to follow. They all highlight that icon-only controls have inherent accessibility problems. From designers in this project I expect accessible and inclusive design. That said, if there is a strong opposition to using buttons with visible text, I'd then second @t-hamano proposal so that we'll use at least one button with visible text 😄 |
Relevant feedback about this issue has been reported also on #58082 which brings in accessibility considerations too. Please take into consideration feedback and proposal from that issue. |
I agree that the icon as it exists is not ideal and located in an odd place. After reading through #58082 and this issue (55179). I think @afercia is on to something here:
|
I've implemented those two suggestions in #58580. |
Let's leave this open as a place to gather feedback with the above changes in place, especially as we head into beta 1 and inevitably more feedback is likely to come into play with a chance to possibly iterate again or keep as is. |
Moved to "done" for now purely for organizational purposes to focus remaining efforts on items that haven't had attention or that need more urgent iteration. |
I'm no sure I understand the statement that using the settings icon 'aligns with existing user expectations', when the actual users feedback that comes from the FSE Outreach Program reports the opposite. Feedback from other contributors and accessibility specialists also reported that this feature is very little discoverable and an icon isn't sufficient. I'd like to bring in one more consideration. The Font Library is one of the most important new features in WordPress 6.5. It's probably _the most awaited feature in 6.5. I'm not sure that hiding the fonts management behind a small, pretty obscure, icon is ideal from a product management perspective in the first place. One more reason to make this feature more prominent by using a button with visible text. |
Hi folks, |
Think we can just mark this as closed by #58580. If the UI needs more work then I'm sure we'll hear about it in a new issue. |
I'm not sure this issue should be closed, as there's still improvements to consider. #58580 only made improvements when there's no font installed. However, when there are fonts installed, teh UI isn't clear. See screenshot: The 'settings' icon doesn't tell me that's the place to open the Manage fonts modal dialog at all. More importantly, there's still direct feedback from users coming from the FSE Outreach Program that hasn't been addressed yet. In #58082 there's a clear proposal to add a 'Manage all fonts' button at the end of the fonts list that hasn't been addressed. As such, I'm reopening this issue for further consideraiton. |
I noticed one more detail that should be reconsidered and I'd call it an UX flow bug. When a theme doesn't provide any font and there's no custom fonts installed, the 'Manage fonts' icon button opens the modal dialog and shows the 'Library' tab, which is empty. There's no great point in bringing users to an empty tab where they can't do anything. |
Pulling in feedback from the FSE Outreach Program's Final Touches call for testing:
I've heard this from a few folks at this point and it's clear that this feature isn't easily discoverable. Perhaps we could consider it as part of iteration for design details: #53483 but for now I wanted to open an issue to gather this feedback.
The text was updated successfully, but these errors were encountered: