From f194e0f5f2a65d5562b7e0916a195dc38c320fa2 Mon Sep 17 00:00:00 2001 From: Murray Stevenson <50844517+murraystevenson@users.noreply.github.com> Date: Mon, 25 Sep 2023 12:57:46 -0700 Subject: [PATCH] StringPlugValueWidget : Fix `Esc` keypress We were calling a function from the legacy PlugValueWidget API, which is not available for classes using the new API. --- Changes.md | 1 + python/GafferUI/StringPlugValueWidget.py | 2 +- 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/Changes.md b/Changes.md index 8fa36fe5612..cf0d544966e 100644 --- a/Changes.md +++ b/Changes.md @@ -5,6 +5,7 @@ Fixes ----- - GraphEditor : Removed dynamic raster-space sizing of focus icon, as it caused excessive overlap with other nodes at certain zoom levels and on certain high resolution displays (#5435). +- StringPlugValueWidget : Fixed bug handling Esc. 1.2.10.3 (relative to 1.2.10.2) ======== diff --git a/python/GafferUI/StringPlugValueWidget.py b/python/GafferUI/StringPlugValueWidget.py index c0a37770f1c..867c3e4b241 100644 --- a/python/GafferUI/StringPlugValueWidget.py +++ b/python/GafferUI/StringPlugValueWidget.py @@ -142,7 +142,7 @@ def __keyPress( self, widget, event ) : # escape abandons everything if event.key == "Escape" : - self._updateFromPlugs() + self._requestUpdateFromValues() return True elif event.key == "Backspace" : # Allow a 'delete' press with the initial keyboard focus and a