Skip to content

fix(input,select): align field backgrounds and borders, add trigger hover - #125

Merged
tmccoy14 merged 1 commit into
mainfrom
feat/MIC-31
Aug 13, 2026
Merged

fix(input,select): align field backgrounds and borders, add trigger hover#125
tmccoy14 merged 1 commit into
mainfrom
feat/MIC-31

Conversation

@tmccoy14

Copy link
Copy Markdown
Contributor

@tmccoy14
tmccoy14 requested review from Shrinks99 and a lite review from Copilot August 13, 2026 23:39
@tmccoy14 tmccoy14 self-assigned this Aug 13, 2026

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the Input and Select component styling in @eqtylab/equality to improve field border/background consistency and adds a subtle hover border treatment for the select trigger (per MIC-31).

Changes:

  • Updated Select trigger to use the shared field background/border tokens and added hover border mixing for light/dark themes.
  • Updated Input and its wrapper to use bg-background and aligned border radius to rounded-md for consistency.
  • Bumped @eqtylab/equality package version from 3.2.1 to 3.2.2.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
packages/ui/src/components/select/select.module.css Aligns select trigger background/border with shared field tokens and adds hover border styling via mixed color variables.
packages/ui/src/components/input/input.module.css Aligns input + input wrapper background and border radius with the same field styling used elsewhere.
packages/ui/package.json Version bump to publish the styling consistency updates.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@tmccoy14
tmccoy14 merged commit 325ce0a into main Aug 13, 2026
2 checks passed
@tmccoy14
tmccoy14 deleted the feat/MIC-31 branch August 13, 2026 23:46
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