Skip to content

feat(Select): maintain size of select fields after value selection#1899

Merged
jallen-cyber merged 1 commit into
mainfrom
select-size-shift-on-value-select
Jul 15, 2026
Merged

feat(Select): maintain size of select fields after value selection#1899
jallen-cyber merged 1 commit into
mainfrom
select-size-shift-on-value-select

Conversation

@jallen-cyber

Copy link
Copy Markdown
Contributor

📝 Changes

A Select with size="md" or size="lg" shrinks by one size step once a value is selected: md renders like sm, and lg renders like md, while sm is unaffected.
Hardcoding variant="body2" on the selected value overrides the inherited size, causing md and lg to render with the smaller body2 style.
Remove the hardcoded variant="body2" so the selected value inherits the trigger's size

✅ Checklist

Easy UI has certain UX standards that must be met. In general, non-trivial changes should meet the following criteria:

  • Visuals match Design Specs in Figma
  • Stories accompany any component changes
  • Code is in accordance with our style guide
  • Design tokens are utilized
  • Unit tests accompany any component changes
  • TSDoc is written for any API surface area
  • Specs are up-to-date
  • Console is free from warnings
  • No accessibility violations are reported
  • Cross-browser check is performed (Chrome, Safari, Firefox)
  • Changeset is added

Strikethrough any items that are not applicable to this pull request.

@jallen-cyber jallen-cyber requested review from a team as code owners July 15, 2026 18:44

@ralexmatthews ralexmatthews left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

@jallen-cyber jallen-cyber merged commit 04c7aab into main Jul 15, 2026
6 checks passed
@jallen-cyber jallen-cyber deleted the select-size-shift-on-value-select branch July 15, 2026 19:08
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