Skip to content

Match the header's search and notification triggers - #20106

Merged
brandonkelly merged 4 commits into
feature/command-palettefrom
feature/command-palette-header-icons
Oct 28, 2026
Merged

brandonkelly merged 4 commits into
feature/command-palettefrom
feature/command-palette-header-icons

Conversation

@brianjhanson

@brianjhanson brianjhanson commented Oct 26, 2026 •

Copy link
Copy Markdown
Contributor

Description

The header's search and notification triggers now match: both are small plain icon buttons with full-size icons, grouped apart from the user menu. Icon size comes from a new --c-icon-size custom property, which also reaches icons nested inside slotted content.

 craft-icon
-  font-size: 0.8em
+  font-size: var(--c-icon-size, 0.8em)

 craft-button  (ellipsis icons)
-  font-size: 1em
+  --c-icon-size: 1em

 CommandPaletteTrigger  (search, was 0.8em)
+  --c-icon-size: 1em

 cp-notification-center  (bell, was 1.1em on the icon)
-  variant="none", .notification-trigger__icon { font-size: 1.1em }
+  variant="plain" icon, --c-icon-size: 1em

Evidence

  • Before: search 10.08px, bell 15.4px
    header-triggers-before
    After: both 12.6px (1em of the small button)
    header-triggers-after

🤖 Generated with Claude Code

brianjhanson and others added 2 commits October 26, 2026 13:21
Buttons now set it to keep ellipsis icons larger, and it reaches icons nested inside slotted content.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Both are now small plain icon buttons with full-size icons, grouped together apart from the user menu.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@github-actions

github-actions Bot commented Oct 26, 2026 •

Copy link
Copy Markdown

📚 Storybook previews

@craftcms/ui — open Storybook

Changed components:

resources/js — open Storybook

No changed components detected in this Storybook.

@brandonkelly
brandonkelly merged commit da1165f into feature/command-palette Oct 28, 2026
20 checks passed
@brandonkelly
brandonkelly deleted the feature/command-palette-header-icons branch October 28, 2026 01:35
brandonkelly added a commit that referenced this pull request Oct 28, 2026
@github-actions github-actions Bot mentioned this pull request Oct 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants