Skip to content

fix: remove seat input spinner controls#119

Merged
asizikov merged 1 commit into
mainfrom
asizikov/copy-pr-8-commit
May 21, 2026
Merged

fix: remove seat input spinner controls#119
asizikov merged 1 commit into
mainfrom
asizikov/copy-pr-8-commit

Conversation

@asizikov
Copy link
Copy Markdown
Member

@asizikov asizikov commented May 21, 2026

Summary

Remove native number input spinner controls from seat fields so custom styling remains consistent across browsers.

Changes

Commit Change
fix: remove seat input spinner controls Hides browser-provided spinner affordances on seat inputs while keeping the existing numeric input behavior.
  • Adds a reusable no-spin-number input style for WebKit and Firefox.
  • Applies it to the Copilot seats input.

Testing

  • npm run build
  • npm run lint
  • npm run test

Checklist

  • Specs lint passes (npm run lint)
  • Tests updated if applicable (N/A)
  • Documentation updated if applicable (N/A)

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings May 21, 2026 07:29
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Removes native browser spinner controls from the seat-count <input type="number"> fields so the custom styling renders consistently across browsers.

Changes:

  • Adds a reusable CSS rule (.no-spin-number) in theme.css to hide number input spinner controls in WebKit and Firefox.
  • Applies the new class to the Copilot Business/Enterprise seat inputs and adds inputMode="numeric".
Show a summary per file
File Description
src/views/UsersView.tsx Applies the new spinner-hiding class to the seat count inputs and sets inputMode="numeric".
src/theme.css Introduces global CSS selectors to remove number input spinner controls when .no-spin-number is present.

Copilot's findings

Tip

Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

  • Files reviewed: 2/2 changed files
  • Comments generated: 1

Comment thread src/views/UsersView.tsx
@asizikov asizikov merged commit a4a20f6 into main May 21, 2026
13 checks passed
@asizikov asizikov deleted the asizikov/copy-pr-8-commit branch May 21, 2026 07:33
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.

3 participants