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

              Ascending order - Click to sort in descending order
              Brad Kroeger created issue -
              Brad Kroeger made changes -
              Field Original Value New Value
              Link This issue blocks ICE-1996 [ ICE-1996 ]
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #14824 Mon Sep 17 16:11:34 MDT 2007 ted.goddard inserting row selector hidden field into first TD (ICE-2109)
              Files Changed
              Commit graph MODIFY /icefaces/trunk/icefaces/component/src/com/icesoft/faces/component/ext/renderkit/TableRenderer.java
              Hide
              Permalink
              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
              Permalink
              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.
              Ted Goddard made changes -
              Assignee Adnan Durrani [ adnan.durrani ]
              Ted Goddard made changes -
              Fix Version/s 1.7DR#1 [ 10100 ]
              Hide
              Permalink
              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.
              Adnan Durrani made changes -
              Status Open [ 1 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Ken Fyten made changes -
              Fix Version/s 1.6.2 [ 10111 ]
              Ken Fyten made changes -
              Resolution Fixed [ 1 ]
              Status Resolved [ 5 ] Reopened [ 4 ]
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #15013 Tue Oct 23 16:33:46 MDT 2007 adnan.durrani Backporting the fix of ICE-2109 to the branch 1.6 (Problem with ice:rowSelector Component in Safari)
              Files Changed
              Commit graph MODIFY /icefaces/branches/icefaces-1.6/icefaces/component/src/com/icesoft/faces/component/ext/renderkit/TableRenderer.java
              Hide
              Permalink
              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
              Adnan Durrani made changes -
              Status Reopened [ 4 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Ken Fyten made changes -
              Fix Version/s 1.7 [ 10080 ]
              Ken Fyten made changes -
              Status Resolved [ 5 ] Closed [ 6 ]
              Assignee Adnan Durrani [ adnan.durrani ]

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved: