Skip to content
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

Replace Star Icon with Pin Icon for 'Pin to Toolbar' #66969

Closed
jayclydeTags opened this issue Nov 13, 2024 · 5 comments
Closed

Replace Star Icon with Pin Icon for 'Pin to Toolbar' #66969

jayclydeTags opened this issue Nov 13, 2024 · 5 comments
Labels
General Interface Parts of the UI which don't fall neatly under other labels. Needs Design Feedback Needs general design feedback.

Comments

@jayclydeTags
Copy link

Currently, the 'Pin to Toolbar' feature uses a star icon to represent pinning a Comment in this case.

Image

I propose using a pin icon instead, as it more accurately represents the action of pinning.

Image

  • A pin icon is a more widely recognized symbol for pinning items.
  • It would provide a more intuitive user experience.
@jayclydeTags
Copy link
Author

also remove the extra space here:

Image

@Mamaduka Mamaduka added General Interface Parts of the UI which don't fall neatly under other labels. Needs Design Feedback Needs general design feedback. labels Nov 13, 2024
@Mamaduka
Copy link
Member

cc @WordPress/gutenberg-design

@jasmussen
Copy link
Contributor

The extra space definitely seems like a regression.

Pinning and unpinning seems broken in a few ways for the moment: every single item that can be pinned, should also have an item in the "More" menu, otherwise you can't open it on mobile. That doesn't seem part of the API at the moment, where it seems arbitrary whether a sidebar honors this or not.

As far as star or pin, I'd still go with star for now. There's a larger question about pinning and unpinning that seems more important to fix in the near term, some larger pieces to improve, so just changing the star isn't going to fix that.

@t-hamano
Copy link
Contributor

Regarding the extra space, it was fixed in #66677.

@Mamaduka
Copy link
Member

Thanks, @t-hamano!

In that case, we can close this issue. As @jasmussen pointed out, changing the Star to Pin icon doesn't improve UX.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
General Interface Parts of the UI which don't fall neatly under other labels. Needs Design Feedback Needs general design feedback.
Projects
None yet
Development

No branches or pull requests

4 participants