-
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
Hide empty paragraph block placeholder when it’s the only block when zoomed-out #65474
Comments
The question at hand seems to be: if a block is in it's placeholder state / empty should it be displayed when |
Good point, it's probably the same for a bunch of other blocks, too, e.g. the empty Tag line block that also nudges folks to start typing 🤔 |
@getdave @andrewserong since we're eventually going to bring edit mode into zoom out view - see #65702 - then this would not be a problem since the content in placeholder mode is simply editable? |
I'm testing #65702 here; it's still a rough case: CleanShot.2024-09-30.at.10.17.17.mp4 |
@richtabor I don't understand 😄 how is the roughness related to this issue. This issue is "should the placeholder still be visible if the paragraph is empty" with the implication arond other placeholders ... The poor UI in the recording is something else, am I wrong? |
The original issue was:
I re-tested this and found
Testing videosScreen.Capture.on.2024-10-01.at.11-16-06.1.mp4Screen.Capture.on.2024-10-01.at.11-16-44.mp4Issues I noticed above in @richtabor's testing video:
Anything else I missed? Can we close this one out or do we want to change the way the placeholder behaves? |
I don’t think leaving the placeholder is a good idea if it’s the only block on the page (an empty page/post) while zoomed out. Otherwise, it’s easy to get into that odd experience I shared above. Does that empty paragraph stay in place when a pattern is inserted from this view? |
Based on that I'd say this is the new requirement. Don't show paragraph block placeholder if
LMK if that is not correct. Thanks 🙇 |
Yup! |
Code-wise - placeholders are hard coded into the various blocks. Example: placeholder={ placeholder || __( 'Heading' ) } You can set the Anyone else got a suggestion for how we could go about implementing this? |
Could one option be to use CSS to hide the placeholders, if that's all we need to do (rather than changing the wording)? There's already one case where we set opacity to gutenberg/packages/block-library/src/paragraph/editor.scss Lines 13 to 19 in 1a29c02
I guess the challenge would be if we have the right selectors available for a CSS condition that matches the situation where we want the placeholder to be hidden? |
But will we also remove the block (the empty paragraph) or will the empty paragraph show up when we exit zoom out? I can see it as a source of frustration already. I am really suspicious of these just in time tweaks that hide stuff. It seems there is an underlying UX issue to solve and we're obscuring it here. |
This seems odd, and would advise against it.
I agree. It would be ideal to take a breath, step back, and re-assess Zoom Out in the context of its purpose and intent. |
This Issue appears to be blocked with no clear consensus. We are rapidly approaching WP 6.7 Beta 3. I'd say the underlying Issue is perhaps that we don't have a good initial state for Zoom Out when the editor is empty. By default the paragraph blocks acts as the default placeholder for all editors. It's clearly designed with post content editing in mind which is why it doesn't play well in Zoom Out. The solution seems to be that if the template is empty we should show a specific placeholder, perhaps with instructional text encouraging you to drag and drop content. Given we are now in Beta we can't add new features to Zoom Out so we need a decision on whether this is sufficiently important to warrant the "fix" (as suggested above) or we defer a solution until 6.8. What do you think @richtabor? Also @kevin940726 @ndiego @colorful-tones as Editor Leads for 6.7. |
I would also like to add that this may be well not solved for 6.7 - it's not such a big problem that there is a placeholder there. Nothing else is editable in 6.7 when zoomed out. My 2 cents. |
The current placeholder situation is not great, but it is also not mission-critical, in my opinion. There are more pressing Zoom Out issues that need solving in this tight time window, so I don't recommend we spend time on this one. |
I'm adding this to the WP 6.7 project board. Although, it sounds like it'll just get the Punt to 6.8 status. |
I would vote to punt this to 6.8 👍 |
Sounds good, thanks @getdave |
After some discussion the new requirement here seems to be:
Description
When zoom out mode is active in the editor / site editor, empty paragraph blocks continue to display their
Type / to choose a block
placeholder state, even though they cannot be selected.Should this be hidden in zoom out mode?
Step-by-step reproduction instructions
Screenshots, screen recording, code snippet
In the following screenshot, the empty paragraph block there cannot be selected, but its placeholder is still visible. Should it be hidden?
Expected
Don't show paragraph block placeholder if
The text was updated successfully, but these errors were encountered: