Skip to content

fix(react-spinbutton): fix read-only functionality#35761

Open
dmytrokirpa wants to merge 2 commits intomicrosoft:masterfrom
dmytrokirpa:fix/react-spinnbutton-readonly
Open

fix(react-spinbutton): fix read-only functionality#35761
dmytrokirpa wants to merge 2 commits intomicrosoft:masterfrom
dmytrokirpa:fix/react-spinnbutton-readonly

Conversation

@dmytrokirpa
Copy link
Contributor

Previous Behavior

When the readOnly prop is applied to SpinButton, users are unable to enter a new value using the up or down buttons. However, pressing the up or down arrow buttons or keyboard keys still changes the value.

New Behavior

When the readOnly prop is applied to SpinButton, the up and down buttons are disabled, and pressing the up or down keys on the keyboard has no effect.

Related Issue(s)

@github-actions
Copy link

📊 Bundle size report

Package & Exports Baseline (minified/GZIP) PR Change
react-components
react-components: entire library
1.291 MB
322.941 kB
1.291 MB
322.957 kB
68 B
16 B
react-spinbutton
SpinButton
33.569 kB
11.049 kB
33.637 kB
11.067 kB
68 B
18 B
Unchanged fixtures
Package & Exports Size (minified/GZIP)
react-components
react-components: Button, FluentProvider & webLightTheme
70.397 kB
19.96 kB
react-components
react-components: Accordion, Button, FluentProvider, Image, Menu, Popover
237.306 kB
68.52 kB
react-components
react-components: FluentProvider & webLightTheme
43.612 kB
14.022 kB
react-portal-compat
PortalCompatProvider
8.386 kB
2.624 kB
react-timepicker-compat
TimePicker
108.15 kB
35.693 kB
🤖 This report was generated against 15159bafa8fed4f264c0df9f49879e8e28f4cbb3

@github-actions
Copy link

Pull request demo site: URL

@@ -0,0 +1,7 @@
{

Choose a reason for hiding this comment

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

🕵🏾‍♀️ visual changes to review in the Visual Change Report

vr-tests-react-components/Positioning 1 screenshots
Image Name Diff(in Pixels) Image Type
vr-tests-react-components/Positioning.Positioning end.chromium.png 745 Changed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: SpinButton up/down buttons active when readOnly is set

1 participant

Comments