bootstack 0.4.0__tar.gz → 0.4.1__tar.gz

{bootstack-0.4.0/src/bootstack.egg-info → bootstack-0.4.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: bootstack
-Version: 0.4.0
+Version: 0.4.1
 Summary: Build desktop GUI apps in pure Python and ship them as a standalone executable — built on Tk, no Tkinter boilerplate.
 Author-email: Israel Dryer <israel.dryer@gmail.com>
 License-Expression: MIT

{bootstack-0.4.0 → bootstack-0.4.1}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "bootstack"
-version = "0.4.0"
+version = "0.4.1"
 description = "Build desktop GUI apps in pure Python and ship them as a standalone executable — built on Tk, no Tkinter boilerplate."
 readme = "README.md"
 requires-python = ">=3.12"
@@ -114,7 +114,7 @@ where = ["src"]
 ]
 
 [tool.bumpversion]
-current_version = "0.4.0"
+current_version = "0.4.1"
 parse = "(?P<major>\\d+)\\.(?P<minor>\\d+)\\.(?P<patch>\\d+)((?P<pre>a|b|rc)(?P<pre_n>\\d+))?"
 serialize = [
     "{major}.{minor}.{patch}{pre}{pre_n}",

{bootstack-0.4.0 → bootstack-0.4.1}/src/bootstack/_core/signal_binding.py

@@ -157,7 +157,8 @@ def create_signal(default_value: Any) -> SignalBinding:
         0.0
     """
     from bootstack.signals import Signal
-    signal = Signal(default_value)
+    # (str, set) mirrors Signal._empty_value()'s own rule
+    signal = Signal(default_value, allow_empty=isinstance(default_value, (str, set)))
     return SignalBinding(
         signal=signal,
         variable=signal.var,

{bootstack-0.4.0 → bootstack-0.4.1}/src/bootstack/signals/signal.py

@@ -108,6 +108,7 @@ class Signal(Generic[T]):
         self._name = name or f"SIG{next(self._cnt)}"
         self._master: tk.Misc | None = master
         self._allow_empty = allow_empty
+
         if dtype is not None:
             if not isinstance(dtype, type):
                 raise TypeError(
@@ -132,6 +133,11 @@ class Signal(Generic[T]):
                 "A Signal that starts empty has no value to take its type from, so "
                 "it must declare one: bs.Signal(None, allow_empty=True, dtype=date)."
             )
+        elif value is None and not allow_empty:
+            raise TypeError(
+                "A Signal that starts empty needs allow_empty=True and a dtype "
+                "or it can never hold any other value."
+            )
         else:
             self._type = type(value)
             self._object_mode = not self._is_tk_native(value)
@@ -354,12 +360,12 @@ class Signal(Generic[T]):
         """
         Set the signal to a new value and notify subscribers.
 
+        In general, the value must match the signals types, however,
+        an `int` may be set on a `float`-typed signal (it is widened), and
+        `None` is accepted on a signal declared `allow_empty=True`.
+
         Args:
-            value: The new value. Must match the signal's type, except that an
-                `int` may be set on a `float`-typed signal (it is widened), and
-                that `None` is accepted on a signal declared `allow_empty=True`,
-                where it becomes that signal's empty value — `''` when the value
-                lives in a widget's variable, `None` otherwise.
+            value: The new value. Must match the signal's type.
 
         Raises:
             TypeError: If the value type does not match the signal's type (and is
@@ -367,12 +373,11 @@ class Signal(Generic[T]):
                 signal that was not declared able to be empty.
         """
         if value is None:
-            # A signal whose type IS NoneType has always taken None (map() makes
-            # one whenever the transform returns None for the first value seen).
             if not self._allow_empty and self._type is not type(None):
                 raise TypeError(
-                    f"Expected {self._type.__name__}, got NoneType. Pass "
-                    f"allow_empty=True to Signal() if this value can be empty."
+                    f"Expected {self._type.__name__}, got NoneType. If you created "
+                    f"this signal, then add `allow_empty=True`, otherwise set the "
+                    f"value directly instead of clearing it."
                 )
             value = self._empty_value()
         else:
@@ -420,10 +425,6 @@ class Signal(Generic[T]):
         Raises:
             TypeError: If the signal was not declared able to be empty.
         """
-        # Through set(None), not set(_empty_value()): the empty of a realized
-        # str signal is '', which is a valid str, so passing it here walked
-        # straight past the allow_empty check and let any realized text signal
-        # be cleared without the declaration (#390 round 2, finding 3).
         self.set(None)
 
     def map(self, transform: Callable[[T], U]) -> 'Signal[U]':

{bootstack-0.4.0 → bootstack-0.4.1}/src/bootstack/widgets/_impl/_parts/spinnerentry_part.py

@@ -159,6 +159,28 @@ class SpinnerEntryPart(ValidationMixin, Spinbox):
 
         self._prev_change_text = text
         self.event_generate('<<Input>>', data=InputEvent(text=text))
+        self._commit_if_not_editing()
+
+    def _commit_if_not_editing(self):
+        """Re-derive the committed value for a change the user did not type (#482)."""
+        try:
+            if self.focus_get() is self:
+                return
+        except (TclError, KeyError):
+            return
+        self._reparse()
+
+    def _reparse(self) -> bool:
+        """Re-derive `_value` from the display text; False if it will not parse."""
+        s = self.get().strip()
+        if s == '':
+            self._value = None if self._allow_blank else self._value
+            return True
+        try:
+            self._value = s if self._value_format is None else self._fmt.parse(s, self._value_format)
+        except ValueError:
+            return False  # keep prior value on parse failure
+        return True
 
     def _check_if_changed(self):
         """Emit <<Change>> event if parsed value changed since focus-in."""
@@ -355,17 +377,8 @@ class SpinnerEntryPart(ValidationMixin, Spinbox):
 
     def commit(self):
         """Parse display text, update value, and normalize display (called on FocusOut/Return)."""
-        s = self.get().strip()
-
-        # Parse once
-        if s == '':
-            self._value = None if self._allow_blank else self._value
-        else:
-            try:
-                self._value = s if self._value_format is None else self._fmt.parse(s, self._value_format)
-            except ValueError:
-                # Keep prior value on parse failure
-                return
+        if not self._reparse():
+            return
 
         # Format the value for display
         new_text = self._format_value(self._value)

{bootstack-0.4.0 → bootstack-0.4.1}/src/bootstack/widgets/_impl/_parts/textentry_part.py

@@ -7,9 +7,9 @@ from bootstack.widgets._impl.primitives.entry import Entry
 from bootstack.widgets._impl.mixins import ValidationMixin
 from bootstack.widgets._impl.mixins.configure_mixin import configure_delegate
 
-#: Sentinel marking "no argument passed" for the combined get/set accessors below.
-#: `None` cannot serve as that sentinel — it is a real value meaning "empty", and
-#: using it to mean "get" made `value(None)` a silent no-op instead of clearing.
+# Sentinel marking "no argument passed" for the combined get/set accessors below.
+# `None` cannot serve as that sentinel — it is a real value meaning "empty", and
+# using it to mean "get" made `value(None)` a silent no-op instead of clearing.
 _UNSET: Any = object()
 
 
@@ -174,6 +174,30 @@ class TextEntryPart(ValidationMixin, Entry):
                 return
         self._prev_change_text = text
         self.event_generate('<<Input>>', data=InputEvent(text=text))
+        self._commit_if_not_editing()
+
+    def _commit_if_not_editing(self):
+        """Re-derive the committed value for a change the user did not type (#482)."""
+        if self._showing_placeholder:
+            return
+        try:
+            if self.focus_get() is self:
+                return
+        except (TclError, KeyError):
+            return
+        self._reparse()  # value only; this runs inside the variable's write trace
+
+    def _reparse(self) -> bool:
+        """Re-derive `_value` from the display text; False if it will not parse."""
+        s = self.get().strip()
+        if s == '':
+            self._value = None if self._allow_blank else self._value
+            return True
+        try:
+            self._value = s if self._value_format is None else self._fmt.parse(s, self._value_format)
+        except ValueError:
+            return False  # keep prior value on parse failure
+        return True
 
     def _check_if_changed(self):
         """Emit <<Change>> event if parsed value changed since focus-in."""
@@ -325,17 +349,8 @@ class TextEntryPart(ValidationMixin, Entry):
         """Parse display text, update value, and normalize display (called on FocusOut/Return)."""
         if self._showing_placeholder:
             return
-        s = self.get().strip()
-
-        # parse once
-        if s == '':
-            self._value = None if self._allow_blank else self._value
-        else:
-            try:
-                self._value = s if self._value_format is None else self._fmt.parse(s, self._value_format)
-            except ValueError:
-                # keep prior value on parse failure
-                return
+        if not self._reparse():
+            return
 
         # Format the value for display
         new_text = self._format_value(self._value)

{bootstack-0.4.0 → bootstack-0.4.1}/src/bootstack/widgets/_impl/composites/textarea/core.py

@@ -352,10 +352,9 @@ class _MultilineCore(tk.Frame):
             self.value = text
 
     def _on_signal_change(self, new_value) -> None:
-        if new_value is not None:
-            text = str(new_value)
-            if self._signal_text() != text:
-                self._apply_signal_text(text)
+        text = str(new_value or "")
+        if self._signal_text() != text:
+            self._apply_signal_text(text)
 
     # ── dirty tracking ────────────────────────────────────────────────────
 

{bootstack-0.4.0 → bootstack-0.4.1}/src/bootstack/widgets/textarea.py

@@ -227,6 +227,8 @@ class TextArea(PublicWidgetBase):
         Args:
             text: Text to insert at the current cursor position.
         """
+        if self._internal._showing_placeholder:
+            self._internal._hide_placeholder()
         self._internal._core.insert("insert", text)
 
     def append(self, text: str) -> None:
@@ -238,6 +240,8 @@ class TextArea(PublicWidgetBase):
         Args:
             text: Text to add at the end of the content.
         """
+        if self._internal._showing_placeholder:
+            self._internal._hide_placeholder()
         self._internal._core.insert("end-1c", text)
 
     def select_all(self) -> None:

{bootstack-0.4.0 → bootstack-0.4.1/src/bootstack.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: bootstack
-Version: 0.4.0
+Version: 0.4.1
 Summary: Build desktop GUI apps in pure Python and ship them as a standalone executable — built on Tk, no Tkinter boilerplate.
 Author-email: Israel Dryer <israel.dryer@gmail.com>
 License-Expression: MIT