← 输入框|github.com|刷新
Skip to content

web: drop team icon borders and widen breadcrumb dropdowns - #27

Merged
yekta merged 1 commit into
masterfrom
style/team-icon-borderless
Oct 17, 2026
Merged

yekta merged 1 commit into
masterfrom
style/team-icon-borderless

Conversation

@yekta

@yekta yekta commented Oct 17, 2026

Copy link
Copy Markdown
Contributor

Team icons no longer have a border. Without it, they can use the same sizes as regular icons instead of custom overrides.

  • Team icon: the border is gone. Placeholder icons with no image, or whose image failed to load, now use bg-border instead of bg-card.
  • Icon sizes: breadcrumb triggers and dropdown and sheet items no longer pass a classNameIcon override, so team icons use the default icon sizes. The trigger icon is now size-4.5. The team icon in the logo link (and its loading placeholder) goes from size-6.5 to size-6, the same size as the logo.
  • Avatar picker: the hover overlay in team settings is now inset-0, since there is no border left to inset it inside.
  • Breadcrumb dropdowns: all breadcrumb dropdowns are sm:w-[min(16rem,var(--available-width))], so they are wider on sm and up but never wider than the available space.
  • Breadcrumb spacing: the trigger gap goes from gap-2 to gap-1.5. The single and double chevrons go from -ml-1 to -ml-0.5 so the spacing stays even.

- Team icons no longer have a border. Placeholder icons without an image now use bg-border, so they can use the same sizes as regular icons.
- Breadcrumb triggers and dropdown items no longer pass custom icon size overrides. The logo link team icon now matches the logo at size-6.
- Breadcrumb dropdowns are 16rem wide on sm and up, capped at the available width.
- The breadcrumb trigger gap goes from gap-2 to gap-1.5, and the chevrons move from -ml-1 to -ml-0.5 to keep the spacing even.
@yekta
yekta merged commit c15a105 into master Oct 17, 2026
8 checks passed
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.

1 participant