ICEfaces
  1. ICEfaces
  2. ICE-9455

ace:dateTimeEntry stops working in editable ace:dataTable

    Details

    • Assignee Priority:
      P1
    • Salesforce Case Reference:
    • Workaround Exists:
      Yes
    • Workaround Description:
      Do not use h:panelGrid in this circumstance.

      Description

      Instructions to reproduce:

      1. Click to edit a table row
      2. Select a date from calendar
      3. Click the commandLink titled 'Click Me'
      4. Try selecting another date. The ace:dateTimeEntry will no longer popup

      There are no suspicious errors in the server logs and focus is attempting to be applied to the calendar:
      [window] persisted focus for element "j_idt9:genAnnounceTable:0:fromDateInput_input". The issue is caused by the following h:panelGrid. The table is located inside index.xhtml. Test case attached.

                      <h:panelGrid>
      <h:panelGroup>
      <h:form>
      <ui:include src="./index.xhtml" />
      </h:form>
      </h:panelGroup>
      </h:panelGrid>

        Activity

        Hide
        yip.ng added a comment -

        Deryk's solution works! Thanks so much to Deryk.

        Show
        yip.ng added a comment - Deryk's solution works! Thanks so much to Deryk. screenshot-04.png video
        Hide
        Deryk Sinotte added a comment -

        With the supplied test case, adding the extra panelGroup around the panelGrid solved the problem in my environment.

        Show
        Deryk Sinotte added a comment - With the supplied test case, adding the extra panelGroup around the panelGrid solved the problem in my environment.
        Hide
        Mark Collette added a comment -

        Deryk, can you please attach the output of using domDiff=debugAB while reproducing this issue, without the extra panelGroup fix.

        Show
        Mark Collette added a comment - Deryk, can you please attach the output of using domDiff=debugAB while reproducing this issue, without the extra panelGroup fix.
        Hide
        Deryk Sinotte added a comment -

        Attaching log of debugAB

        Show
        Deryk Sinotte added a comment - Attaching log of debugAB
        Hide
        Mark Collette added a comment -

        From the test case's debugAB, we can see that the issue of TEXT nodes being aggregated to cause a DOM difference was a key issue for the full update. So, I updated to the newest icefaces3 trunk and ICEfaces EE 3.3.0 P01 code, and tested for that with them and they came up clean. So no lingering code issues, and an app work-around for the old code shows this is resolved.

        Show
        Mark Collette added a comment - From the test case's debugAB, we can see that the issue of TEXT nodes being aggregated to cause a DOM difference was a key issue for the full update. So, I updated to the newest icefaces3 trunk and ICEfaces EE 3.3.0 P01 code, and tested for that with them and they came up clean. So no lingering code issues, and an app work-around for the old code shows this is resolved.

          People

          • Assignee:
            yip.ng
            Reporter:
            Tyler Johnson
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: