[royale-asjs] branch develop updated: jewel-numericstepper: remove CHANGE and use ENTER and FOCUSOUT (with 'change' If we have minimun to 10, using change event on input always return 10)

[carlosrovira]

This is an automated email from the ASF dual-hosted git repository.

carlosrovira pushed a commit to branch develop
in repository https://gitbox.apache.org/repos/asf/royale-asjs.git


The following commit(s) were added to refs/heads/develop by this push:
     new 44ba2ed  jewel-numericstepper: remove CHANGE and use ENTER and 
FOCUSOUT (with 'change' If we have minimun to 10, using change event on input 
always return 10)
44ba2ed is described below

commit 44ba2edbd136148f68f51b4c938a91566d8fe0a5
Author: Carlos Rovira <carlos.rov...@apache.org>
AuthorDate: Sat Dec 5 15:42:44 2020 +0100

    jewel-numericstepper: remove CHANGE and use ENTER and FOCUSOUT (with 
'change' If we have minimun to 10, using change event on input always return 10)
---
 .../royale/jewel/beads/controllers/NumericStepperController.as   | 9 ++++++++-
 .../org/apache/royale/jewel/beads/views/NumericStepperView.as    | 1 -
 2 files changed, 8 insertions(+), 2 deletions(-)

diff --git 
a/frameworks/projects/Jewel/src/main/royale/org/apache/royale/jewel/beads/controllers/NumericStepperController.as
 
b/frameworks/projects/Jewel/src/main/royale/org/apache/royale/jewel/beads/controllers/NumericStepperController.as
index 0e556f5..8be3e65 100644
--- 
a/frameworks/projects/Jewel/src/main/royale/org/apache/royale/jewel/beads/controllers/NumericStepperController.as
+++ 
b/frameworks/projects/Jewel/src/main/royale/org/apache/royale/jewel/beads/controllers/NumericStepperController.as
@@ -90,7 +90,14 @@ package org.apache.royale.jewel.beads.controllers
                        // value. ideally, we should either set the input to 
accept only
                        // numeric values or, barring that, reject non-numeric 
entries. we
                        // cannot do that right now however.
-                       input.addEventListener(Event.CHANGE, 
inputChangeHandler);
+
+                       // we can't use CHANGE, since it will change 'value' on 
each key stroke
+                       // moreover, let's say we have a minimun of 10, 
entering '5', will make the value jump to 10
+                       input.addEventListener("enter", inputChangeHandler);
+                       COMPILE::JS
+                       {
+                               input.element.addEventListener('blur', 
inputChangeHandler);
+                       }
                        
                        // listen for change events on the spinner so the value 
can be updated as
                        // as resizing the component
diff --git 
a/frameworks/projects/Jewel/src/main/royale/org/apache/royale/jewel/beads/views/NumericStepperView.as
 
b/frameworks/projects/Jewel/src/main/royale/org/apache/royale/jewel/beads/views/NumericStepperView.as
index 8a6c309..a9ac4e6 100644
--- 
a/frameworks/projects/Jewel/src/main/royale/org/apache/royale/jewel/beads/views/NumericStepperView.as
+++ 
b/frameworks/projects/Jewel/src/main/royale/org/apache/royale/jewel/beads/views/NumericStepperView.as
@@ -96,7 +96,6 @@ package org.apache.royale.jewel.beads.views
                        (value as IParent).addElement(spinnerview.decrement as 
IChild);
                        
                        var model:IRangeModel = 
_strand.getBeadByType(IRangeModel) as IRangeModel;
-                       trace(model, model.value);
                        spinner.value = model.value;
                        input.text = String(model.value);