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

Search Block “Optional placeholder” text takes on nav block underline #34275

Closed
2 tasks done
annezazu opened this issue Aug 25, 2021 · 3 comments · Fixed by #36345
Closed
2 tasks done

Search Block “Optional placeholder” text takes on nav block underline #34275

annezazu opened this issue Aug 25, 2021 · 3 comments · Fixed by #36345
Assignees
Labels
[Block] Navigation Affects the Navigation Block [Block] Search Affects the Search Block - used to display a search field CSS Styling Related to editor and front end styles, CSS-specific issues. [Status] In Progress Tracking issues with work in progress [Type] Bug An existing feature does not function as intended

Comments

@annezazu
Copy link
Contributor

Description

When assigning the underline styling to the overall navigation block, the "optional placeholder" text adopts the underline likely due to some CSS bleeding.

Step-by-step reproduction instructions

  1. Add a navigation block to the site editor.
  2. Add a search block.
  3. Select the overall navigation block and, in the sidebar settings, select the underline option.
  4. Notice that the "optional placeholder" text shows underlined text.

Screenshots, screen recording, code snippet

underline.issue.mov

Environment info

  • WordPress 5.8
  • Gutenberg 11.3
  • Desktop with MacOS

Pre-checks

  • I have searched the existing issues.
  • I have tested with all plugins deactivated except Gutenberg.
@annezazu annezazu added [Type] Bug An existing feature does not function as intended [Block] Navigation Affects the Navigation Block [Block] Search Affects the Search Block - used to display a search field CSS Styling Related to editor and front end styles, CSS-specific issues. labels Aug 25, 2021
@github-actions github-actions bot added the [Status] In Progress Tracking issues with work in progress label Aug 30, 2021
@jasmussen
Copy link
Contributor

Good ticket. I have a "fix" in #34379 that we might not want to merge. Another option is to simply remove text decoration support from the navigation block entirely!

@jasmussen
Copy link
Contributor

After exploring the fix in #34379, conversation in the comments suggested that underline due to its unique nature on a technical level needs to be individually applied to each menu item, lest it will be inherited all the way down the stack.

Whether that also means we move the underline control to each menu item, or whether we can/should provide a nav-block global toggle is up for conversation. But it means this one needs a different fix.

@annezazu
Copy link
Contributor Author

annezazu commented Feb 7, 2022

Noting that text decoration has been removed from the navigation block and is not in 5.9 (was looping back on the state of things): #37963 Going to leave this open until the above draft PR is merged/work continues to address this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Block] Navigation Affects the Navigation Block [Block] Search Affects the Search Block - used to display a search field CSS Styling Related to editor and front end styles, CSS-specific issues. [Status] In Progress Tracking issues with work in progress [Type] Bug An existing feature does not function as intended
Projects
None yet
3 participants