ICEfaces
  1. ICEfaces
  2. ICE-2109

Problem with ice:rowSelector Component in Safari

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.6.1
    • Fix Version/s: 1.6.2, 1.7DR#1, 1.7
    • Component/s: None
    • Labels:
      None
    • Environment:
      Safari on Windows
      Sarari on iPhone
    • Workaround Exists:
      Yes
    • Workaround Description:
      A workaround could be attempted that would imitate the ice:rowSelector component with ice:commandLinks and CSS tweaks.

      Description

      The taxi mobile demo application located at repo\applications\taxi-mobile has a problem with Safari on Windows and Safari on the iPhone.

      When viewing as a cabbie (taxicab.iface) you see an ice:dataTable with all the cab requests. This table is successfully receiving ajax push updates when a style change is made, but if a request is added or removed from the dataTable (the number of table items change), the ajax push causes the dataTable not to render.

        Activity

        Hide
        Ted Goddard added a comment -

        A very similar bug, ICE-1259 was fixed in the past. Including an <input> within the <table> or <tbody> is not valid HTML and Safari objects. The intended fix is to insert the hidden field for this component within the first <td>.

        Show
        Ted Goddard added a comment - A very similar bug, ICE-1259 was fixed in the past. Including an <input> within the <table> or <tbody> is not valid HTML and Safari objects. The intended fix is to insert the hidden field for this component within the first <td>.
        Hide
        Ted Goddard added a comment -

        Adnan, please review the change and test general row selector functionality.

        Show
        Ted Goddard added a comment - Adnan, please review the change and test general row selector functionality.
        Hide
        Adnan Durrani added a comment -

        Ted's fix working fine. I didn't find any side effect.

        Show
        Adnan Durrani added a comment - Ted's fix working fine. I didn't find any side effect.
        Hide
        Adnan Durrani added a comment -

        Fix backported to 1.6 branch. revision : 15013

        Show
        Adnan Durrani added a comment - Fix backported to 1.6 branch. revision : 15013

          People

          • Assignee:
            Unassigned
            Reporter:
            Brad Kroeger
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: