Merge pull request #7535 from Budibase/reduce-onchange-event-frequency
Only fire onchange events from form fields when values change
This commit is contained in:
commit
1edf3b2082
11 changed files with 21 additions and 22 deletions
|
@ -48,8 +48,8 @@
|
||||||
}
|
}
|
||||||
|
|
||||||
const handleChange = e => {
|
const handleChange = e => {
|
||||||
fieldApi.setValue(e.detail)
|
const changed = fieldApi.setValue(e.detail)
|
||||||
if (onChange) {
|
if (onChange && changed) {
|
||||||
onChange({ value: e.detail })
|
onChange({ value: e.detail })
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
|
@ -28,8 +28,8 @@
|
||||||
}
|
}
|
||||||
|
|
||||||
const handleChange = e => {
|
const handleChange = e => {
|
||||||
fieldApi.setValue(e.detail)
|
const changed = fieldApi.setValue(e.detail)
|
||||||
if (onChange) {
|
if (onChange && changed) {
|
||||||
onChange({ value: e.detail })
|
onChange({ value: e.detail })
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
|
@ -17,8 +17,8 @@
|
||||||
let fieldApi
|
let fieldApi
|
||||||
|
|
||||||
const handleChange = e => {
|
const handleChange = e => {
|
||||||
fieldApi.setValue(e.detail)
|
const changed = fieldApi.setValue(e.detail)
|
||||||
if (onChange) {
|
if (onChange && changed) {
|
||||||
onChange({ value: e.detail })
|
onChange({ value: e.detail })
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
|
@ -268,7 +268,7 @@
|
||||||
|
|
||||||
// Skip if the value is the same
|
// Skip if the value is the same
|
||||||
if (!skipCheck && fieldState.value === value) {
|
if (!skipCheck && fieldState.value === value) {
|
||||||
return
|
return false
|
||||||
}
|
}
|
||||||
|
|
||||||
// Update field state
|
// Update field state
|
||||||
|
|
|
@ -37,8 +37,8 @@
|
||||||
|
|
||||||
const handleChange = e => {
|
const handleChange = e => {
|
||||||
const value = parseValue(e.detail)
|
const value = parseValue(e.detail)
|
||||||
fieldApi.setValue(value)
|
const changed = fieldApi.setValue(value)
|
||||||
if (onChange) {
|
if (onChange && changed) {
|
||||||
onChange({ value })
|
onChange({ value })
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
|
@ -47,8 +47,8 @@
|
||||||
}
|
}
|
||||||
|
|
||||||
const handleChange = e => {
|
const handleChange = e => {
|
||||||
fieldApi.setValue(e.detail)
|
const changed = fieldApi.setValue(e.detail)
|
||||||
if (onChange) {
|
if (onChange && changed) {
|
||||||
onChange({ value: e.detail })
|
onChange({ value: e.detail })
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
|
@ -44,8 +44,8 @@
|
||||||
}
|
}
|
||||||
|
|
||||||
const handleChange = e => {
|
const handleChange = e => {
|
||||||
fieldApi.setValue(e.detail)
|
const changed = fieldApi.setValue(e.detail)
|
||||||
if (onChange) {
|
if (onChange && changed) {
|
||||||
onChange({ value: e.detail })
|
onChange({ value: e.detail })
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
|
@ -34,8 +34,8 @@
|
||||||
)
|
)
|
||||||
|
|
||||||
const handleChange = e => {
|
const handleChange = e => {
|
||||||
fieldApi.setValue(e.detail)
|
const changed = fieldApi.setValue(e.detail)
|
||||||
if (onChange) {
|
if (onChange && changed) {
|
||||||
onChange({ value: e.detail })
|
onChange({ value: e.detail })
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
|
@ -84,8 +84,8 @@
|
||||||
}
|
}
|
||||||
|
|
||||||
const handleChange = value => {
|
const handleChange = value => {
|
||||||
fieldApi.setValue(value)
|
const changed = fieldApi.setValue(value)
|
||||||
if (onChange) {
|
if (onChange && changed) {
|
||||||
onChange({ value })
|
onChange({ value })
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
|
@ -90,8 +90,8 @@
|
||||||
}
|
}
|
||||||
|
|
||||||
const handleChange = e => {
|
const handleChange = e => {
|
||||||
fieldApi.setValue(e.detail)
|
const changed = fieldApi.setValue(e.detail)
|
||||||
if (onChange) {
|
if (onChange && changed) {
|
||||||
onChange({ value: e.detail })
|
onChange({ value: e.detail })
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
|
@ -16,8 +16,8 @@
|
||||||
let fieldApi
|
let fieldApi
|
||||||
|
|
||||||
const handleChange = e => {
|
const handleChange = e => {
|
||||||
fieldApi.setValue(e.detail)
|
const changed = fieldApi.setValue(e.detail)
|
||||||
if (onChange) {
|
if (onChange && changed) {
|
||||||
onChange({ value: e.detail })
|
onChange({ value: e.detail })
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
@ -29,7 +29,6 @@
|
||||||
{disabled}
|
{disabled}
|
||||||
{validation}
|
{validation}
|
||||||
{defaultValue}
|
{defaultValue}
|
||||||
{onChange}
|
|
||||||
type={type === "number" ? "number" : "string"}
|
type={type === "number" ? "number" : "string"}
|
||||||
bind:fieldState
|
bind:fieldState
|
||||||
bind:fieldApi
|
bind:fieldApi
|
||||||
|
|
Loading…
Reference in a new issue