Skip to content
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

Video block settings: Poster image label needs spacing #35785

Closed
sabernhardt opened this issue Oct 19, 2021 · 0 comments · Fixed by #37747
Closed

Video block settings: Poster image label needs spacing #35785

sabernhardt opened this issue Oct 19, 2021 · 0 comments · Fixed by #37747
Assignees
Labels
[Block] Video Affects the Video Block [Status] In Progress Tracking issues with work in progress [Type] Enhancement A suggestion for improvement.

Comments

@sabernhardt
Copy link
Contributor

Description

In the Video block settings, the Poster image label has no bottom margin or padding. The margin was intentionally increased from 4px to 8px in #20754, but that has been missing since WordPress 5.6.

Should the label use StyledLabel instead of BaseControl.VisualLabel?

<BaseControl.VisualLabel>
{ __( 'Poster image' ) }
</BaseControl.VisualLabel>

(originally reported on Trac)

Step-by-step reproduction instructions

  1. Open the post editor.
  2. Add a Video block, and upload or select a video.
  3. Make sure the settings sidebar is open and scroll to view the Poster image control with no spacing between the label and the button.
  4. Select a poster image to see the focus outline obscure part of the label text.

Screenshots, screen recording, code snippet

Before selecting a poster image:

poster image label in sidebar

With the poster image:

poster image label with Replace button and its focus outline

(screenshots taken in my 5.9 trunk installation, without the plugin)

Environment info

  • WP 5.8.1, with and without Gutenberg 11.7.0
  • Firefox 93
  • Windows 10 20H2 Home

Please confirm that you have searched existing issues in the repo.

Yes

Please confirm that you have tested with all plugins deactivated except Gutenberg.

Yes

@Mamaduka Mamaduka added [Block] Video Affects the Video Block [Type] Enhancement A suggestion for improvement. labels Oct 20, 2021
@github-actions github-actions bot added the [Status] In Progress Tracking issues with work in progress label Jan 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Block] Video Affects the Video Block [Status] In Progress Tracking issues with work in progress [Type] Enhancement A suggestion for improvement.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants