ICEfaces
  1. ICEfaces
  2. ICE-1712

<ice:selectInputDate/> fails to set focus after date is selected from popup calender

          Details

          • Type: Bug Bug
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: 1.6
          • Fix Version/s: 1.6.1
          • Component/s: ICE-Components
          • Labels:
            None
          • Environment:
            Win XP / JDK 1.4 / Tomcat 5.5.17 / Liferay 4.3.0-RC1

            Description

            I happened to notice this because I saw an error in the ICEfaces SHIFT+CTRL+T window:

            10:31:10 error [window] : Failed to set focus on [_sample_icefaces_sun_WAR_sampleicefacessunportlet_INSTANCE_iBR8__id1:_sample_icefaces_sun_WAR_sampleicefacessunportlet_INSTANCE_iBR8__id3:dateOfBirth_1574_calendar][object Error]

            See attached portlet in order to reproduce.

            Thanks,
            Neil

            ---

            This issue is happening without portlets also, as reported in the Forum:

            When calling requestFocus on instance of selectInputDate component I am getting javascript error in the debug window.

            Code:

             12:54:49 error [window] : Failed to set focus on [calendarEditForm:sendWhenDate]
             TypeError: id.asExtendedElement().focus is not a function
             

              Activity

              Ascending order - Click to sort in descending order
              Hide
              Permalink
              Neil Griffin added a comment -

              Forgot to mention that I first noticed this problem as of trunk revision 14108 (6/11/2007), which is the version of the JARs you will find in the WEB-INF/lib folder of the attached portlet.

              Show
              Neil Griffin added a comment - Forgot to mention that I first noticed this problem as of trunk revision 14108 (6/11/2007), which is the version of the JARs you will find in the WEB-INF/lib folder of the attached portlet.
              Hide
              Permalink
              Adnan Durrani added a comment -

              Fix revision # 14713

              The requestFocus() method has been fixed as well as the default behavior added to the selectInputDate component so the input text will get a focus by default after selecting a date from pop-up calendar.

              Show
              Adnan Durrani added a comment - Fix revision # 14713 The requestFocus() method has been fixed as well as the default behavior added to the selectInputDate component so the input text will get a focus by default after selecting a date from pop-up calendar.
              Hide
              Permalink
              Neil Griffin added a comment -

              This appears to be broken again in ICEfaces 1.7RC1

              Show
              Neil Griffin added a comment - This appears to be broken again in ICEfaces 1.7RC1
              Hide
              Permalink
              Neil Griffin added a comment -

              To be more precise, when I select a date from the popup calendar, the input focus is placed on the little popup handle icon, rather than on the input field. I suppose this could be considered correct behavior, but from an end user (usability) perspective, I would think that the date that gets inserted into the text field would be selected in a "select all" kind of manner. Then pressing tab again would send me to the next field.

              Show
              Neil Griffin added a comment - To be more precise, when I select a date from the popup calendar, the input focus is placed on the little popup handle icon, rather than on the input field. I suppose this could be considered correct behavior, but from an end user (usability) perspective, I would think that the date that gets inserted into the text field would be selected in a "select all" kind of manner. Then pressing tab again would send me to the next field.
              Hide
              Permalink
              Neil Griffin added a comment -

              Note that there is no "failed to set focus" error in the ICEfaces debug window – so this bug is indeed fixed, I guess I'm questioning the correctness of setting input focus on the calendar popup button/handle rather than the text field.

              Show
              Neil Griffin added a comment - Note that there is no "failed to set focus" error in the ICEfaces debug window – so this bug is indeed fixed, I guess I'm questioning the correctness of setting input focus on the calendar popup button/handle rather than the text field.

                People

                • Assignee:
                  Unassigned
                  Reporter:
                  Neil Griffin
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  0 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: