From 8d8fa6a835fee91f70b17426fa3e9f29a73dea2b Mon Sep 17 00:00:00 2001 From: CrushedPixel Date: Thu, 30 Jul 2015 11:00:44 +0200 Subject: [PATCH] Be more reasonable in GuiKeyframeRepository with user inputs, not replacing the keyframeSetList's elements every time the Keyframe Preset's name changes and therefore not calling the Listener which set the nameInput's value every time. This leads to a smoother user experience regarding the nameInput Text Field. --- .../crushedpixel/replaymod/gui/GuiKeyframeRepository.java | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/src/main/java/eu/crushedpixel/replaymod/gui/GuiKeyframeRepository.java b/src/main/java/eu/crushedpixel/replaymod/gui/GuiKeyframeRepository.java index bbdcfb7f..7b17b4f8 100644 --- a/src/main/java/eu/crushedpixel/replaymod/gui/GuiKeyframeRepository.java +++ b/src/main/java/eu/crushedpixel/replaymod/gui/GuiKeyframeRepository.java @@ -226,10 +226,11 @@ public class GuiKeyframeRepository extends GuiScreen implements GuiReplayOverlay saveOnQuit(); super.keyTyped(typedChar, keyCode); } - if(currentKeyframeSet != null) { - currentKeyframeSet.setName(nameInput.getText()); - keyframeSetList.replaceElement(keyframeSetList.getSelectionIndex(), currentKeyframeSet); + if(currentKeyframeSet != null && !currentKeyframeSet.getName().equals(nameInput.getText().trim())) { + currentKeyframeSet.setName(nameInput.getText().trim()); } + + super.keyTyped(typedChar, keyCode); } private void saveOnQuit() {