-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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 Editing: UI Parity #21245
Comments
Device preview added in #21309. |
PR up for top toolbar and focus mode in #22537 |
Pulling a discussion out of: #22539 (comment) for more visibility. cc @youknowriad & @epiqueras For the time being, we have accepted that duplicating some code between edit-post and edit-site is acceptable. We don't want to make an abstraction too early which doesn't make sense in the long run. However, when looking into porting the "tools" section of edit-post into edit-site, I realized there is a significant amount of code to copy over. For two reasons, I wonder if we should not duplicate some of this stuff until we come up with a better abstraction:
In the future, since we would need to un-duplicate it anyways, I think it would mostly just be wasted time. This doesn't apply to everything. For example, focus mode and the block toolbar in the header are both primarily implemented in the block-editor package, so the majority of the code is reused between edit-site and edit-post anyways. Additionally, the code editor will need to be updated to work better with multiple entities. But some of this other stuff seems like a lot of work for not a lot of benefit at the time being. This leads me to wonder how much of the "basic UI parity" features should be duplicated before we come up with a good abstraction. I don't think "all of them" makes sense because there would be so much to basically just copy over and hook in. And then we have two copies of everything floating around and we aren't any closer to having a better abstraction. |
A |
I don't think they are that important at this stage. And I agree with you that it would be a waste of time if they are duplicated now. |
This looks related: #21430 |
@mtias Perhaps you could update the initial checkbox item "keyboard shortcuts" in the initial comment to reflect the fact that there have been at least two separate issues raised and closed without a fix or any sign of a plan to fix them. |
I see this checked off currently:
However, when I look at the experience today, I only see the option to export items but don't see a specific help option in place. Since FSE is so new, it would be advantageous to have a link to support docs built into the interface. Is this something that can be added ahead of 5.9? |
I started working on Keyboard Shortcut help modal. Unfortunately, it's a bit late for the 5.9, but we should backport it into a minor release. |
@annezazu, do we have a documentation URL for the "Help" button? |
This will eventually be the URL: https://wordpress.org/support/article/site-editor/ This will provide a similar experience as the Post Editor which links to https://wordpress.org/support/article/wordpress-editor/. |
@annezazu I'm out of the loop because of the holidays, does that mean the project agreed to keep using the term "site editor" ? |
It's only going to be called "Editor" under Appearance but I'm not quite sure how else to position this. If we reuse the current https://wordpress.org/support/article/wordpress-editor/ doc and add in FSE related content, it'll likely confuse many of the folks who are just using the Post Editor. The best middle ground I could provide was by creating a separate doc for the Site Editor since that matches what was done with the template editor. Open to ideas/thoughts though around how best to proceed here :) In time, I imagine all of this will turn into just the WordPress editor but, for now, we need some in between solutions. Here's the documentation tracking issue here: WordPress/Documentation-Issue-Tracker#93 |
These are some of the features that are lacking in
edit-site
at the moment compared to regular editor.The text was updated successfully, but these errors were encountered: