ICEfaces
  1. ICEfaces
  2. ICE-11016

ensure mobile input components comply with ICEfaces 4 standard ajax submit

          Details

          • Affects:
            Compatibility/Configuration

            Description

            the current mobi:dateSpinner and mobi:timeSpinner components do a singleSubmit when users enter a value into the input field. THe only time this should happen is if the component uses ace:ajax (similar to all other input components in ICEfaces 4).

              Activity

              Ascending order - Click to sort in descending order
              Judy Guglielmin created issue -
              Judy Guglielmin made changes -
              Field Original Value New Value
              Assignee Judy Guglielmin [ judy.guglielmin ]
              Judy Guglielmin made changes -
              Fix Version/s EE-4.1.0.BETA [ 13072 ]
              Judy Guglielmin made changes -
              Fix Version/s EE-4.1.0.GA [ 12171 ]
              Hide
              Permalink
              Judy Guglielmin added a comment -

              rev. 48678. commit rev 48677 for ICE-10876 fixes this for mobi:dateSpinner and mobi:flipswitch was already compliant.

              Show
              Judy Guglielmin added a comment - rev. 48678. commit rev 48677 for ICE-10876 fixes this for mobi:dateSpinner and mobi:flipswitch was already compliant.
              Judy Guglielmin made changes -
              Status Open [ 1 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #48678 Mon May 02 13:11:03 MDT 2016 judy.guglielmin ICE-11016 enter when useNative now does not do submit unless ace:ajax tag is present. ICE-10876 already fixes this problem for dateSpinner. mobi:flipswitch is only other input component with 4.0, and does not submit unless ace:ajax so already compliant
              Files Changed
              Commit graph MODIFY /icefaces4/trunk/icefaces/mobi/component/src/org/icefaces/mobi/component/timespinner/TimeSpinnerRenderer.java
              Hide
              Permalink
              Liana Munroe added a comment -

              Verified ICEfaces EE-4.1.0 r 48691, Tomcat 8, IE 11, 10, FF 43, Chrome 50

              Show
              Liana Munroe added a comment - Verified ICEfaces EE-4.1.0 r 48691, Tomcat 8, IE 11, 10, FF 43, Chrome 50
              Ken Fyten made changes -
              Affects Compatibility/Configuration [ 10002 ]
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #48773 Thu May 12 10:48:43 MDT 2016 judy.guglielmin ICE-11016 enter when useNative now does not do submit unless ace:ajax tag is present. Add ace:ajax to showcase example page
              Files Changed
              Commit graph MODIFY /icefaces4/trunk/icefaces/samples/showcase/showcase/src/main/webapp/resources/examples/mobi/datespinner/date-example.xhtml
              Hide
              Permalink
              Liana Munroe added a comment -

              Reopened for submit issues with date/Time Spinner showcase demo ICEfaces EE-4.1.0 RC build 4

              Show
              Liana Munroe added a comment - Reopened for submit issues with date/Time Spinner showcase demo ICEfaces EE-4.1.0 RC build 4
              Liana Munroe made changes -
              Resolution Fixed [ 1 ]
              Status Resolved [ 5 ] Reopened [ 4 ]
              Hide
              Permalink
              Judy Guglielmin added a comment -

              investigating the iPad timeSpinner submit issue. If there are more issues with dateSpinner or timeSpinner than this one, please let me know as I don't see any issues at all with dateSpinner.

              Show
              Judy Guglielmin added a comment - investigating the iPad timeSpinner submit issue. If there are more issues with dateSpinner or timeSpinner than this one, please let me know as I don't see any issues at all with dateSpinner.
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #48785 Fri May 13 11:27:13 MDT 2016 judy.guglielmin ICE-11016 touch devices use own event as per TimeSpinner method getDefaultEventName.
              Files Changed
              Commit graph MODIFY /icefaces4/trunk/icefaces/mobi/component/resources/org.icefaces.component.datespinner/datespinner.js
              Commit graph MODIFY /icefaces4/trunk/icefaces/mobi/component/resources/org.icefaces.component.timespinner/timespinner.js
              Commit graph MODIFY /icefaces4/trunk/icefaces/mobi/component/src/org/icefaces/mobi/component/timespinner/TimeSpinnerRenderer.java
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #48790 Fri May 13 14:09:06 MDT 2016 judy.guglielmin ICE-11016 remove ability for hidden field to submit when value is null (when another component commits and there is no inputValue). Not using hidden value any longer, since ability to use native widget as well as developed widget > Timespinner now just writes the call directly (rather than calling a method in component js.
              Files Changed
              Commit graph MODIFY /icefaces4/trunk/icefaces/mobi/component/src/org/icefaces/mobi/component/datespinner/DateSpinnerRenderer.java
              Commit graph MODIFY /icefaces4/trunk/icefaces/mobi/component/resources/org.icefaces.component.timespinner/timespinner.js
              Commit graph MODIFY /icefaces4/trunk/icefaces/mobi/component/src/org/icefaces/mobi/component/timespinner/TimeSpinnerRenderer.java
              Hide
              Permalink
              Judy Guglielmin added a comment -

              when useNative is true, the dateSpinner was submitting in the showcase example just by opening the timeSpinner widget (useNative) and causing the date to be reset to today's date. (double decode!).
              Still a known issue on some Android devices, that the change when input type="date" (dateSpinner useNative="true"), the onchange will not be fired until the field loses focus.

              Show
              Judy Guglielmin added a comment - when useNative is true, the dateSpinner was submitting in the showcase example just by opening the timeSpinner widget (useNative) and causing the date to be reset to today's date. (double decode!). Still a known issue on some Android devices, that the change when input type="date" (dateSpinner useNative="true"), the onchange will not be fired until the field loses focus.
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #48799 Fri May 13 17:02:05 MDT 2016 judy.guglielmin ICE-11016 ensure focus moves to proper field so that the opening of useNative="true" does not fire a change on the other (focussed) element. Some android devices require field to be left to fire "onchange" is known issue at this time.
              Files Changed
              Commit graph MODIFY /icefaces4/trunk/icefaces/mobi/component/resources/org.icefaces.component.datespinner/datespinner.js
              Commit graph MODIFY /icefaces4/trunk/icefaces/mobi/component/src/org/icefaces/mobi/component/datespinner/DateSpinnerRenderer.java
              Commit graph MODIFY /icefaces4/trunk/icefaces/mobi/component/src/org/icefaces/mobi/component/timespinner/TimeSpinnerRenderer.java
              Ken Fyten made changes -
              Status Reopened [ 4 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Hide
              Permalink
              Liana Munroe added a comment -

              Verified ICEfaces EE-4.1.0 RC build 5. Tomcat8, ipad with ios 9.3.1, android 6.0.1.
              Note** the date and time spinners do not act identically in the showcase demo when using the ipad or android device.
              After picking a date you must blur away from the date picker in order to have the date submitted and value updated. The time spinner updates the value without having to blur away.

              Show
              Liana Munroe added a comment - Verified ICEfaces EE-4.1.0 RC build 5. Tomcat8, ipad with ios 9.3.1, android 6.0.1. Note** the date and time spinners do not act identically in the showcase demo when using the ipad or android device. After picking a date you must blur away from the date picker in order to have the date submitted and value updated. The time spinner updates the value without having to blur away.
              Ken Fyten made changes -
              Issue Type Bug [ 1 ] Improvement [ 4 ]
              Hide
              Permalink
              Liana Munroe added a comment - - edited

              Verified ICEfaces EE-4.1.1.BETA build 2. Tomcat8, ipad with ios 9.3.1, android 6.0.1.
              Note** the date and time spinners do not act identically in the showcase demo when using the ipad or android device.
              After picking a date you must blur away from the date picker in order to have the date submitted and value updated. The time spinner updates the value without having to blur away.
              With the ios 9.3.1 ipad there is no "done" or "set" button on the native spinner which is why you must tap away from the date spinner to set the date.

              Show
              Liana Munroe added a comment - - edited Verified ICEfaces EE-4.1.1.BETA build 2. Tomcat8, ipad with ios 9.3.1, android 6.0.1. Note** the date and time spinners do not act identically in the showcase demo when using the ipad or android device. After picking a date you must blur away from the date picker in order to have the date submitted and value updated. The time spinner updates the value without having to blur away. With the ios 9.3.1 ipad there is no "done" or "set" button on the native spinner which is why you must tap away from the date spinner to set the date.
              Hide
              Permalink
              Ken Fyten added a comment -

              We'll consider the discrepancy between these two components behaviour a Known Issue on Chrome since this appears to be a Chrome issue with the native widgets, not much we can do about it.

              Show
              Ken Fyten added a comment - We'll consider the discrepancy between these two components behaviour a Known Issue on Chrome since this appears to be a Chrome issue with the native widgets, not much we can do about it.
              Ken Fyten made changes -
              Fix Version/s 4.2.BETA [ 13091 ]
              Fix Version/s 4.2 [ 12870 ]
              Ken Fyten made changes -
              Status Resolved [ 5 ] Closed [ 6 ]

                People

                • Assignee:
                  Judy Guglielmin
                  Reporter:
                  Judy Guglielmin
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  3 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: