laipeng/reflex
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

remove prop debounce_timeout from input and textarea (#1645)

Browse Source
This commit is contained in:
danik292 2023-08-21 23:44:59 +02:00 committed by GitHub
parent d884b7ba96
commit af08e06569
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 12 additions and 14 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
reflex/components/forms/input.py
View File

@ -50,9 +50,6 @@ class Input(ChakraComponent):
# "lg" | "md" | "sm" | "xs"
size: Var[str]
# Time in milliseconds to wait between end of input and triggering on_change
debounce_timeout: Var[int]
def _get_imports(self) -> imports.ImportDict:
return imports.merge_imports(
super()._get_imports(),
@ -84,12 +81,14 @@ class Input(ChakraComponent):
Returns:
The component.
"""
if isinstance(props.get("value"), Var) and props.get("on_change"):
if (
isinstance(props.get("value"), Var) and props.get("on_change")
) or "debounce_timeout" in props:
# Currently default to 50ms, which appears to be a good balance
debounce_timeout = props.pop("debounce_timeout", 50)
# create a debounced input if the user requests full control to avoid typing jank
return DebounceInput.create(
super().create(*children, **props),
# Currently default to 50ms, which appears to be a good balance
debounce_timeout=props.get("debounce_timeout", 50),
super().create(*children, **props), debounce_timeout=debounce_timeout
)
return super().create(*children, **props)

13
reflex/components/forms/textarea.py
View File

@ -43,9 +43,6 @@ class TextArea(ChakraComponent):
# "outline" | "filled" | "flushed" | "unstyled"
variant: Var[str]
# Time in milliseconds to wait between end of input and triggering on_change
debounce_timeout: Var[int]
def get_controlled_triggers(self) -> Dict[str, Var]:
"""Get the event triggers that pass the component's value to the handler.
@ -71,11 +68,13 @@ class TextArea(ChakraComponent):
Returns:
The component.
"""
if isinstance(props.get("value"), Var) and props.get("on_change"):
if (
isinstance(props.get("value"), Var) and props.get("on_change")
) or "debounce_timeout" in props:
# Currently default to 50ms, which appears to be a good balance
debounce_timeout = props.pop("debounce_timeout", 50)
# create a debounced input if the user requests full control to avoid typing jank
return DebounceInput.create(
super().create(*children, **props),
# Currently default to 50ms, which appears to be a good balance
debounce_timeout=props.get("debounce_timeout", 50),
super().create(*children, **props), debounce_timeout=debounce_timeout
)
return super().create(*children, **props)