-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
Polish design of edit interfaces for meta in page inspector #60291
Comments
Status panel is being worked on in #60456. |
@jameskoster what about the pingbacks & trackbacks in posts? How do they fit with the comments setting? You can see the current checkbox in post editor in |
What about the label of the button though? It can't just say Also there is the possibility that we only have the trackbaacks checkbox available and not the comments, if the post type supports only that.. |
Perhaps it moves to the ellipsis now, then :) It doesn't feel worthy of a main item in the summary panel to me, but happy to hear other opinions about that. |
The only challenge with the ellipsis is whether that check enables the pingbacks/trackbacks, or whether it surfaces the toggle in the panel, ToolsPanel style. I'd agree on the emphasis, pingbacks is a bit of a vestigial thing, and I'd think it even something you set once in your reading settings and then have a default based on that. Honestly we can even surface the control in the panel based on those same reading settings. |
The tools panel pattern here would be a little convoluted I think. The idea was that selecting the menu item basically worked the same as the current checkbox. |
I think we can close this issue now, no? |
There's probably some minor polishing to do, but that can happen in individual issues / PRs 👍 |
As a part of the work in #59689 it would be nice to tidy up the edit interface for page meta data. These UIs looking a little rough around the edges, and the designs in 59689 call for some options like discussion and parent selection to move into the Summary panel.
Here's the current state of affairs:
The text was updated successfully, but these errors were encountered: