Opened 3 weeks ago
Last modified 3 days ago
#49613 reviewing defect (bug)
Twenty Nineteen: Editor style interferes with navigation menu.
Reported by: | Joen | Owned by: | ianbelanger |
---|---|---|---|
Milestone: | 5.5 | Priority: | normal |
Severity: | normal | Version: | |
Component: | Bundled Theme | Keywords: | good-first-bug has-patch |
Focuses: | css | Cc: |
Description
This issue is only present in the latest master version of the Gutenberg plugin.
The following rule is present in the TwentyNineteen editor style:
.wp-block .wp-block { width: 100%; }
This rule was created, if I recall correctly, to adjust the dimensions of nested blocks, and override another rule that set the width of top level blocks to a complex calc rule.
As of https://github.com/WordPress/gutenberg/pull/20729, however, this rule now unintentionally targets navigation menu items, which should show up horizontally in a flow. Specifically, the width of the .wp-block-navigation-link must be auto, or it won't work.
Suggest refactoring the rule above to only target top level blocks in different manner, or use the same rule but apply width: initial; instead, if possible.
Attachments (3)
Change History (10)
#1
@
3 weeks ago
- Focuses css added
- Milestone changed from Awaiting Review to Future Release
- Summary changed from TwentyNineteen: Editor style interferes with navigation menu. to Twenty Nineteen: Editor style interferes with navigation menu.
- Version trunk deleted
Thanks for reporting this @Joen. Do you know when this is planning to be merged into core? Just so we can be ready for it when the time comes.
#2
@
3 weeks ago
The change is likely to land in the plugin for next weeks release. But it is extremely unlikely to be an issue in core until version 5.5.
#3
@
3 weeks ago
- Keywords good-first-bug added
- Milestone changed from Future Release to 5.5
Great! Thanks for the quick response. I'll mark it for 5.5
.
#4
@
6 days ago
Thanks, @Joen. 49613.diff changes that rule to initial
instead of 100%
, and it appears to work fine. Tested with a bunch of Group, Cover, and Columns blocks, and haven't noticed any issues.
Before:
After:
How it should look