-
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
Site Editor: Add View Link #57153
Site Editor: Add View Link #57153
Conversation
Size Change: +343 B (0%) Total Size: 1.71 MB
ℹ️ View Unchanged
|
Flaky tests detected in 081b21b. 🔍 Workflow run URL: https://github.com/WordPress/gutenberg/actions/runs/7248139222
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
When "Show button text labels" is enabled, when I open any page in the Site Editor in edit mode and reload the browser, the button text will switch from "View Post" to "View Page". Is there any way to get around this?
807012f432307710708acb6892cae5a0.mp4
Co-authored-by: Aki Hamano <[email protected]>
Maybe I don't understand exactly, but does it make sense to refer to |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, thank you! I'm approving since the issue @t-hamano mentions already exists in trunk and we can update easily.
If we really want to defer here we could just use:
label={ label }
The view_item
label already default is ‘View Post’ / ‘View Page’.
Co-authored-by: Aki Hamano <[email protected]>
Related #52632
What?
This PR continues the work on the great unification between post and site editors. This PR moves the ViewLink component from the edit-post package to the editor package and reuse the same panel in the site editor as well. This means that for post types that are viewable in the frontend, there will be a link to "view the page" in the frontend in the header of the editor in the site editor, similar to the post editor.
Testing instructions