ICEfaces
  1. ICEfaces
  2. ICE-8267

ace:dateTimeEntry - regains focus after manually editing input field

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: EE-3.0.0.GA, 3.1.0.BETA1, 3.1.0.BETA2
    • Fix Version/s: 3.1.0.RC1, 3.1, EE-3.0.0.GA_P01
    • Component/s: ACE-Components
    • Labels:
      None
    • Environment:
      ICEfaces 3.1.0 BETA2-PREV
    • Assignee Priority:
      P3

      Description

      After manually editing the date value in the dateTimeEntry input field and then focusing away from to a different input text field, the focus is returned to the dateTimeEntry input field and the calendar popup is shown.

        Activity

        Hide
        Arran Mccullough added a comment -

        Attached test case that shows this issue.

        Steps to reproduce:

        • Open welcomeICEfaces.jsf
        • Focus into the second input field and select a date through the popup calendar.
        • Focus back to the input field and modify one of the date values.
        • Once the value has been changed, using the mouse click in the first input field.
        • Focus will goto the input field and then immediately goto the dateTimeEntry input field, and therefore opening the calendar.
        Show
        Arran Mccullough added a comment - Attached test case that shows this issue. Steps to reproduce: Open welcomeICEfaces.jsf Focus into the second input field and select a date through the popup calendar. Focus back to the input field and modify one of the date values. Once the value has been changed, using the mouse click in the first input field. Focus will goto the input field and then immediately goto the dateTimeEntry input field, and therefore opening the calendar.
        Hide
        yip.ng added a comment - - edited

        Nothing to do with component. Caused by framework focus and submit management. See screenshot-01.png.

        The best a component can do is set the remembered focus id to empty on blur, and let the framework figure out where to focus next. The framework function used to do that – setFoucs() (misleading name, it doesn't actually set focus) – seems to have disappeared from the current builds. (Got a "function not defined" error now. See http://jira.icesoft.org/secure/attachment/14594/screenshot-04.png) It was still working recently and is still in one of the framework JS source files.

        Show
        yip.ng added a comment - - edited Nothing to do with component. Caused by framework focus and submit management. See screenshot-01.png. The best a component can do is set the remembered focus id to empty on blur, and let the framework figure out where to focus next. The framework function used to do that – setFoucs() (misleading name, it doesn't actually set focus) – seems to have disappeared from the current builds. (Got a "function not defined" error now. See http://jira.icesoft.org/secure/attachment/14594/screenshot-04.png ) It was still working recently and is still in one of the framework JS source files.
        Hide
        Ken Fyten added a comment -

        This has been fixed now with a component change.

        Show
        Ken Fyten added a comment - This has been fixed now with a component change.
        Hide
        yip.ng added a comment -

        Done.

        Modified: C:\svn\ossrepo\icefaces3\branches\icefaces-3.0.x-maintenance\icefaces\ace\component\resources\icefaces.ace\datetimeentry\datetimeentry.js
        Completed: At revision: 29847

        Show
        yip.ng added a comment - Done. Modified: C:\svn\ossrepo\icefaces3\branches\icefaces-3.0.x-maintenance\icefaces\ace\component\resources\icefaces.ace\datetimeentry\datetimeentry.js Completed: At revision: 29847

          People

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

            Dates

            • Created:
              Updated:
              Resolved: