-
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
Try: Swap pencil icon for drawer icon in template details panel #54190
Conversation
Warning: Type of PR label error To merge this PR, it requires exactly 1 label indicating the type of PR. Other labels are optional and not being checked here.
Read more about Type labels in Gutenberg. |
Size Change: +215 B (0%) Total Size: 1.62 MB
ℹ️ View Unchanged
|
Flaky tests detected in e37d455. 🔍 Workflow run URL: https://github.com/WordPress/gutenberg/actions/runs/6085638309
|
I see the intent, but I don't know that "Close sidebar & edit" is going to be better than simply "Edit", which would also be inclusive of mobile where this wouldn't be a sidebar that closes. |
Just chiming in that I've seen this feedback as well with the FSE Outreach Program. I'd like to see this iterated upon and am glad to see this PR. |
I think we can find a better edit icon, but I would avoid the "Edit" button label, we had that in earlier mockups and it does not scale to the detail pages that feature both ellipses, edit icons, and .. what was the third? |
That might be worth a try. |
Appreciate the effort. I personally think it's too early still to remove the edit button, or to put it in the menu. |
Yeah, we're running out of options... Overall I think I prefer the text button. The scalability issues can potentially be reduced by using the |
What about the smaller edit icon? Without the __ under the pencil, perhaps it's less "Edit title". |
What?
In the template details panel there is an icon button (
pencil
) which invokes the full-screen editor on click. This PR replacespencil
withdrawerLeft
/drawerRight
(according tois_RTL()
).Why?
I've seen feedback that the pencil icon looks like a 'rename' affordance for the title.
Testing Instructions
Before
After
I'm not sure how successful this is so I've only updated the template details panel for now. Perhaps there are other solutions we might consider. What do y'all think?