ICEfaces
  1. ICEfaces
  2. ICE-9392

REGRESSION: ace:dateTimeEntry - Clicking Today button closes popup

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 3.3, EE-3.3.0.GA
    • Fix Version/s: EE-3.3.0.GA_P01, 4.0.BETA, 4.0
    • Component/s: ACE-Components
    • Labels:
      None
    • Environment:
      All
    • Assignee Priority:
      P1
    • Salesforce Case Reference:

      Description

      With the EE 3.2.0 release and older, clicking the Today button for the ace:dateTimeEntry component would select the current days date and keep the popup opened. With the 3.3.0 and EE 3.3.0 releases this behavior has changed. On click of the Today button it selects the current date but closes the popup.
      1. Case12255Example.zip
        20 kB
        Arran Mccullough
      1. screenshot-01.png
        303 kB

        Activity

        Arran Mccullough created issue -
        Arran Mccullough made changes -
        Field Original Value New Value
        Salesforce Case Reference 5007000000UZj7bAAD
        Hide
        Arran Mccullough added a comment -

        Attached test case source code that shows the issue.

        Show
        Arran Mccullough added a comment - Attached test case source code that shows the issue.
        Arran Mccullough made changes -
        Attachment Case12255Example.zip [ 16103 ]
        Hide
        Ken Fyten added a comment -

        This change in behavior may be related to changes made for ICE-8820.

        Show
        Ken Fyten added a comment - This change in behavior may be related to changes made for ICE-8820 .
        Ken Fyten made changes -
        Summary ace:dateTimeEntry - Clicking Today button closes popup REGRESSION: ace:dateTimeEntry - Clicking Today button closes popup
        Ken Fyten made changes -
        Assignee yip.ng [ yip.ng ]
        Fix Version/s 3.4 [ 10770 ]
        Assignee Priority P1 [ 10010 ]
        Hide
        Ken Fyten added a comment -

        We'll need to add a configuration parameter or attribute to allow this mode of operation, in addition to the current behavior where clicking "Today" updates the inputfield as well. Perhaps a global config param. would be simplest?

        Show
        Ken Fyten added a comment - We'll need to add a configuration parameter or attribute to allow this mode of operation, in addition to the current behavior where clicking "Today" updates the inputfield as well. Perhaps a global config param. would be simplest?
        Ken Fyten made changes -
        Fix Version/s EE-3.3.0.GA_P01 [ 11174 ]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #36630 Wed Jul 10 17:47:23 MDT 2013 yip.ng ICE-9392: REGRESSION: ace:dateTimeEntry - Clicking Today button closes popup.
        Files Changed
        Commit graph MODIFY /icefaces3/trunk/icefaces/ace/component/resources/icefaces.ace/jquery/ui/jquery-ui.js
        Commit graph MODIFY /icefaces3/trunk/icefaces/ace/component/resources/icefaces.ace/datetimeentry/jquery-ui-timepicker-addon.js
        Commit graph MODIFY /icefaces3/trunk/icefaces/ace/component/resources/icefaces.ace/datetimeentry/readme.txt
        Commit graph MODIFY /icefaces3/trunk/icefaces/ace/component/src/org/icefaces/ace/component/datetimeentry/DateTimeEntryMeta.java
        Commit graph MODIFY /icefaces3/trunk/icefaces/ace/component/src/org/icefaces/ace/component/datetimeentry/DateTimeEntryRenderer.java
        Commit graph MODIFY /icefaces3/trunk/icefaces/ace/component/resources/icefaces.ace/jquery/ui/readme.txt
        yip.ng made changes -
        Attachment screenshot-01.png [ 16113 ]
        Hide
        yip.ng added a comment -
        1. Regression of ICE-8820 caused by upgrade of timepicker (jquery-ui-timepicker-addon.js) in ICE-8920. 382 differences. An override function has some new code that bypassed the checking code added to jquery-ui.js for ICE-8820.
        2. New attribute added to control whether Today/Now selects as well: screenshot-01.png.

        M: C:\svn\ossrepo\icefaces3\trunk\icefaces\ace\component\src\org\icefaces\ace\component\datetimeentry\DateTimeEntryMeta.java#36630
        M: C:\svn\ossrepo\icefaces3\trunk\icefaces\ace\component\src\org\icefaces\ace\component\datetimeentry\DateTimeEntryRenderer.java#36630
        M: C:\svn\ossrepo\icefaces3\trunk\icefaces\ace\component\resources\icefaces.ace\datetimeentry\jquery-ui-timepicker-addon.js#36630
        M: C:\svn\ossrepo\icefaces3\trunk\icefaces\ace\component\resources\icefaces.ace\jquery\ui\jquery-ui.js#36630
        M: C:\svn\ossrepo\icefaces3\trunk\icefaces\ace\component\resources\icefaces.ace\datetimeentry\readme.txt#36630
        M: C:\svn\ossrepo\icefaces3\trunk\icefaces\ace\component\resources\icefaces.ace\jquery\ui\readme.txt#36630

        Show
        yip.ng added a comment - Regression of ICE-8820 caused by upgrade of timepicker (jquery-ui-timepicker-addon.js) in ICE-8920 . 382 differences. An override function has some new code that bypassed the checking code added to jquery-ui.js for ICE-8820 . New attribute added to control whether Today/Now selects as well: screenshot-01.png . M: C:\svn\ossrepo\icefaces3\trunk\icefaces\ace\component\src\org\icefaces\ace\component\datetimeentry\DateTimeEntryMeta.java#36630 M: C:\svn\ossrepo\icefaces3\trunk\icefaces\ace\component\src\org\icefaces\ace\component\datetimeentry\DateTimeEntryRenderer.java#36630 M: C:\svn\ossrepo\icefaces3\trunk\icefaces\ace\component\resources\icefaces.ace\datetimeentry\jquery-ui-timepicker-addon.js#36630 M: C:\svn\ossrepo\icefaces3\trunk\icefaces\ace\component\resources\icefaces.ace\jquery\ui\jquery-ui.js#36630 M: C:\svn\ossrepo\icefaces3\trunk\icefaces\ace\component\resources\icefaces.ace\datetimeentry\readme.txt#36630 M: C:\svn\ossrepo\icefaces3\trunk\icefaces\ace\component\resources\icefaces.ace\jquery\ui\readme.txt#36630
        Hide
        yip.ng added a comment -

        M: http:\\10.18.39.25:8888\svn\ossrepo\icefaces-ee\tags\icefaces-ee-3.3.0.GA_P01\icefaces\ace\component\src\org\icefaces\ace\component\datetimeentry\DateTimeEntryMeta.java#36694
        M: http:\\10.18.39.25:8888\svn\ossrepo\icefaces-ee\tags\icefaces-ee-3.3.0.GA_P01\icefaces\ace\component\src\org\icefaces\ace\component\datetimeentry\DateTimeEntryRenderer.java#36694
        M: http:\\10.18.39.25:8888\svn\ossrepo\icefaces-ee\tags\icefaces-ee-3.3.0.GA_P01\icefaces\ace\component\resources\icefaces.ace\datetimeentry\jquery-ui-timepicker-addon.js#36694
        M: http:\\10.18.39.25:8888\svn\ossrepo\icefaces-ee\tags\icefaces-ee-3.3.0.GA_P01\icefaces\ace\component\resources\icefaces.ace\jquery\ui\jquery-ui.js#36694
        M: http:\\10.18.39.25:8888\svn\ossrepo\icefaces-ee\tags\icefaces-ee-3.3.0.GA_P01\icefaces\ace\component\resources\icefaces.ace\jquery\ui\readme.txt#36694
        M: http:\\10.18.39.25:8888\svn\ossrepo\icefaces-ee\tags\icefaces-ee-3.3.0.GA_P01\icefaces\ace\component\resources\icefaces.ace\datetimeentry\readme.txt#36694

        Show
        yip.ng added a comment - M: http:\\10.18.39.25:8888\svn\ossrepo\icefaces-ee\tags\icefaces-ee-3.3.0.GA_P01\icefaces\ace\component\src\org\icefaces\ace\component\datetimeentry\DateTimeEntryMeta.java#36694 M: http:\\10.18.39.25:8888\svn\ossrepo\icefaces-ee\tags\icefaces-ee-3.3.0.GA_P01\icefaces\ace\component\src\org\icefaces\ace\component\datetimeentry\DateTimeEntryRenderer.java#36694 M: http:\\10.18.39.25:8888\svn\ossrepo\icefaces-ee\tags\icefaces-ee-3.3.0.GA_P01\icefaces\ace\component\resources\icefaces.ace\datetimeentry\jquery-ui-timepicker-addon.js#36694 M: http:\\10.18.39.25:8888\svn\ossrepo\icefaces-ee\tags\icefaces-ee-3.3.0.GA_P01\icefaces\ace\component\resources\icefaces.ace\jquery\ui\jquery-ui.js#36694 M: http:\\10.18.39.25:8888\svn\ossrepo\icefaces-ee\tags\icefaces-ee-3.3.0.GA_P01\icefaces\ace\component\resources\icefaces.ace\jquery\ui\readme.txt#36694 M: http:\\10.18.39.25:8888\svn\ossrepo\icefaces-ee\tags\icefaces-ee-3.3.0.GA_P01\icefaces\ace\component\resources\icefaces.ace\datetimeentry\readme.txt#36694
        yip.ng made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Ken Fyten made changes -
        Fix Version/s 4.0 [ 11382 ]
        Ken Fyten made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

          • Assignee:
            yip.ng
            Reporter:
            Arran Mccullough
          • Votes:
            1 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: