Reduce specificity of nav link default padding so global styles apply#76876
Reduce specificity of nav link default padding so global styles apply#76876tellthemachines merged 1 commit intotrunkfrom
Conversation
|
Size Change: +61 B (0%) Total Size: 7.73 MB
ℹ️ View Unchanged
|
|
Flaky tests detected in b32c80b. 🔍 Workflow run URL: https://github.com/WordPress/gutenberg/actions/runs/23676288704
|
MaggieCabrera
left a comment
There was a problem hiding this comment.
The change looks sensible and it tests well. We should revisit the UI for this block to support these controls in another PR (self reminder here kinda)
|
The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the Unlinked AccountsThe following contributors have not linked their GitHub and WordPress.org accounts: @jeghebe. Contributors, please read how to link your accounts to ensure your work is properly credited in WordPress releases. If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message. To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook. |
|
Could this fix be backported to WP 7.0 Beta/RC? |
At this point in the release only fixes for regressions from 6.9 should be merged, and I believe this issue has been around for longer than that. |
…pplied (#76876) Unlinked contributors: jeghebe. Co-authored-by: tellthemachines <isabel_brison@git.wordpress.org> Co-authored-by: MaggieCabrera <onemaggie@git.wordpress.org> Co-authored-by: mrleemon <leemon@git.wordpress.org> Co-authored-by: prasadkarmalkar <prasadkarmalkar@git.wordpress.org> Co-authored-by: 3kori <r1k0@git.wordpress.org>
…pplied (#76876) Unlinked contributors: jeghebe. Co-authored-by: tellthemachines <isabel_brison@git.wordpress.org> Co-authored-by: MaggieCabrera <onemaggie@git.wordpress.org> Co-authored-by: mrleemon <leemon@git.wordpress.org> Co-authored-by: prasadkarmalkar <prasadkarmalkar@git.wordpress.org> Co-authored-by: 3kori <r1k0@git.wordpress.org>
…pplied (#76876) Unlinked contributors: jeghebe. Co-authored-by: tellthemachines <isabel_brison@git.wordpress.org> Co-authored-by: MaggieCabrera <onemaggie@git.wordpress.org> Co-authored-by: mrleemon <leemon@git.wordpress.org> Co-authored-by: prasadkarmalkar <prasadkarmalkar@git.wordpress.org> Co-authored-by: 3kori <r1k0@git.wordpress.org>
What?
Fixes #76306
Default padding added in block library for Navigation Link has higher specificity than global styles, so these don't work.
It was originally added with that specificity to override generic list padding styles that themes might add under
.entry-content li(see here for context). I had a quick look in Veloria and couldn't find any of those, which doesn't mean they don't exist, but it's not super likely themes will add a padding reset tolibecause user agents don't add padding to that element.I guess we could remove it altogether, but in case anyone out there is relying on it this PR just lowers the rule specificity enough that it doesn't interfere with global styles.
Testing Instructions