Commit a61ec64
Remove loose check on non-number controlled inputs. Fix trailing dot issue. (facebook#9584)
* Remove loose check when assigning non-number inputs
This commit removes a check I added when working on number input
issues where we perform a loose check on an input's value before we
assign it. This prevented controlled text inputs from disallowing
numeric text entry.
I also added a DOM fixture text case.
Related issues:
facebook#9561 (comment)
* Use strict equality as a guard before assigning input.value
This commit adds back the guard around assigning the value property to
an input, however it does it using a strict equals. This prevents
validated inputs, like emails and urls from losing the cursor
position.
It also adds associated test fixtures.
* Add copy command after build for interup with surge.sh1 parent e88e47d commit a61ec64
File tree
6 files changed
+192
-311
lines changed- fixtures/dom/src/components
- fixtures
- number-inputs
- src/renderers/dom/client/wrappers
- __tests__
6 files changed
+192
-311
lines changedThis file was deleted.
This file was deleted.
Lines changed: 0 additions & 37 deletions
This file was deleted.
Lines changed: 0 additions & 167 deletions
This file was deleted.
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
226 | 226 | | |
227 | 227 | | |
228 | 228 | | |
229 | | - | |
230 | | - | |
| 229 | + | |
231 | 230 | | |
232 | 231 | | |
233 | 232 | | |
| |||
0 commit comments