Skip to content

fix(NumberField): remove +/- stepper buttons (#9444)#9448

Merged
eliotlim merged 1 commit into
developfrom
fix/number-field-remove-steppers
Jun 4, 2026
Merged

fix(NumberField): remove +/- stepper buttons (#9444)#9448
eliotlim merged 1 commit into
developfrom
fix/number-field-remove-steppers

Conversation

@LoneRifle
Copy link
Copy Markdown
Contributor

Problem and Solution

Closes #9048
Merges #9444 by @dew1997

Copilot AI review requested due to automatic review settings May 20, 2026 01:24
@LoneRifle LoneRifle requested a review from a team as a code owner May 20, 2026 01:24
Copy link
Copy Markdown
Contributor

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 the increment/decrement (“+/-”) stepper controls from the public-facing NumberField to reduce accidental value changes and improve accessibility for assistive-technology users, aligning with the reported usability/a11y issues in #9048.

Changes:

  • Disable NumberInput steppers in NumberField via showSteppers={false}.
  • Add a frontend regression test asserting the stepper buttons are not rendered.

Reviewed changes

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

File Description
apps/frontend/src/templates/Field/Number/NumberField.tsx Disables stepper rendering for the NumberField’s underlying NumberInput.
apps/frontend/src/templates/Field/Number/NumberField.test.tsx Adds a test to ensure increment/decrement stepper buttons are not present.

@eliotlim eliotlim merged commit 0c4c1ff into develop Jun 4, 2026
44 checks passed
@eliotlim eliotlim deleted the fix/number-field-remove-steppers branch June 4, 2026 04:12
@eliotlim eliotlim mentioned this pull request Jun 4, 2026
24 tasks
eliotlim added a commit that referenced this pull request Jun 4, 2026
* chore: add agent context and scratch to gitignore (#9544)

* Merge pull request #9545 from opengovsg/fix/optional-pdf-display

fix: render signature question with empty answer when signature is not captured in PDF

* Merge pull request #9465 from opengovsg/feat/create-form-modal-cutover

feat(mrf-cutover): default CreateFormModal to MRF with storage-mode escape hatch (4/6)

* fix(NumberField): remove +/- stepper buttons (#9444) (#9448)

Co-authored-by: dew1997 <109208984+dew1997@users.noreply.github.com>

* fix: enable line breaks in thank you page message (#9507)

* fix: enable line breaks in thank you page message (#9433)

* fix: lint frontend EndPageBlock (#9509)

Fix frontend lint error in EndPageBlock

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>

---------

Co-authored-by: Bhanu Pratap Singh Rathore <bhanur05@gmail.com>
Co-authored-by: Copilot <198982749+Copilot@users.noreply.github.com>

* Merge pull request #9527 from opengovsg/fix/signature-hover-theme-color

fix(sig): match hover state to form theme color

* fix(table): add default for dropdown columns (#9526)

fix(table): add default for dropdown columns (#9476)

Co-authored-by: Raj Shekar Patha <rajashekarpatha07@gmail.com>

* feat(i18n): add landing payments and not found error pages (#9522)

* feat(i18n): add landing payments and not found error pages (#9497)

feat: add i18n for landing payments and not found error pages

* fix(i18n/payments): remove spurious whitespace

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

* fix(payments): add i18n t hook to `useCallback()`

---------

Co-authored-by: Samuel Tan <126168312+SAMTAN444@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

* chore: bump version to 7.23.0

---------

Co-authored-by: Kevin Foong <55353265+kevin9foong@users.noreply.github.com>
Co-authored-by: scottheng96 <44297674+scottheng96@users.noreply.github.com>
Co-authored-by: LoneRifle <LoneRifle@users.noreply.github.com>
Co-authored-by: dew1997 <109208984+dew1997@users.noreply.github.com>
Co-authored-by: Bhanu Pratap Singh Rathore <bhanur05@gmail.com>
Co-authored-by: Copilot <198982749+Copilot@users.noreply.github.com>
Co-authored-by: Raj Shekar Patha <rajashekarpatha07@gmail.com>
Co-authored-by: Samuel Tan <126168312+SAMTAN444@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
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.

fix(NumberField): remove number field - / + buttons

4 participants