Skip to content

Commit

Permalink
fix: number field format options (#827)
Browse files Browse the repository at this point in the history
Closes #695
  • Loading branch information
ethanalvizo authored Oct 15, 2024
1 parent 93ca388 commit 317e80b
Show file tree
Hide file tree
Showing 3 changed files with 505 additions and 4 deletions.
7 changes: 3 additions & 4 deletions plugins/ui/src/deephaven/ui/components/number_field.py
Original file line number Diff line number Diff line change
@@ -1,20 +1,18 @@
from __future__ import annotations
from typing import Any, Callable
from .types import (
# Events
FocusEventCallable,
KeyboardEventCallable,
# Validation
TextFieldValidationState,
NecessityIndicator,
# Layout
AlignSelf,
CSSProperties,
DimensionValue,
JustifySelf,
LayoutFlex,
Position,
LabelPosition,
NumberFieldFormatOptions,
Alignment,
)
from .basic import component_element
Expand All @@ -27,7 +25,7 @@ def number_field(
decrement_aria_label: str | None = None,
increment_aria_label: str | None = None,
is_wheel_disabled: bool | None = None,
# format_options, # omitted because need to connect it to Deephaven formatting options as well
format_options: NumberFieldFormatOptions | None = None,
is_disabled: bool | None = None,
is_read_only: bool | None = None,
is_required: bool | None = None,
Expand Down Expand Up @@ -186,6 +184,7 @@ def number_field(
decrement_aria_label=decrement_aria_label,
increment_aria_label=increment_aria_label,
is_wheel_disabled=is_wheel_disabled,
format_options=format_options,
is_disabled=is_disabled,
is_read_only=is_read_only,
is_required=is_required,
Expand Down
Loading

0 comments on commit 317e80b

Please sign in to comment.