Needs review
Project:
Drupal core
Version:
11.x-dev
Component:
Claro theme
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
11 Aug 2026 at 13:51 UTC
Updated:
22 Sep 2026 at 13:09 UTC
Jump to comment: Most recent

Comments
Comment #7
dbuzinov commentedI’ve prepared a small fix for the Claro Views UI button spacing issue.
Summary:
This patch fixes the extra spacing in the Views UI action buttons in Claro by increasing the CSS specificity so it overrides the general .button styles.
The root cause was that the action button in the Views tab menu was inheriting the default button spacing from the generic .button styling, while the input-button variant used in the action list was not covered by the more specific selector. I updated the selector to target the specific Views UI button class and reset the margin/box shadow for the input variant.
Files changed:
What changed:
Added a more specific selector for .views-tabs__action-list-button.button
Added a dedicated rule for input.views-tabs__action-list-button.button
Reset margin-block and box-shadow so it matches the expected Claro button styling
This keeps the fix scoped to the Views UI action list without altering the global button styling used elsewhere.
Please review the patch and let me know if further adjustments are needed.
Comment #8
idebr commentedThe regression introduced in #1945262: Introduce "before" and "after" for conditional ordering in library definitions placed css/components/button.css after css/components/views-ui.css. Preferably the regression is fixed by restoring the previous file order for the smallest impact on subthemes
Comment #9
dbuzinov commentedUpdated. The CSS workaround in views-ui.css and views-ui.pcss.css is removed, those files now match main.
The fix is a one-line change in claro.libraries.yml: css/components/button.css moves back to the component CSS category, from theme.
This restores button.css loading before views-ui.css, as it did before #1945262.
Closing my MR. Claro was removed from Drupal core starting with D12 (main), in #3584639. The fix here no longer applies to main, since the affected file no longer exists there.
Comment #11
dbuzinov commentedComment #12
smustgrave commentedAs a recent regression this is still eligible to be fixed in core but claro is already removed in main. So 11.x is the only target here.
Comment #13
dbuzinov commented