ICEfaces
  1. ICEfaces
  2. ICE-5275

compat: showcase rowSelector demo is not working under IE

          Details

          • Type: Bug Bug
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: 2.0-Alpha1
          • Fix Version/s: 2.0-Alpha2, 2.0.0
          • Component/s: ICE-Components
          • Labels:
            None
          • Environment:
            rowSelector

            Description

            compat: showcase rowSelector demo is not working under IE

              Issue Links

              blocks

              New Feature - A new feature of the product, which has yet to be developed. ICE-4458 create component-showcase sample for compat version of ICEfaces-2

              • Major - Major loss of function.
              • Closed

                Activity

                Ascending order - Click to sort in descending order
                Hide
                Permalink
                Judy Guglielmin added a comment -

                required for ICEfaces-2.0 A2 release

                Show
                Judy Guglielmin added a comment - required for ICEfaces-2.0 A2 release
                Hide
                Permalink
                Adnan Durrani added a comment -

                fixed.

                Modified: D:\work\development\head\svn\ossrepo\icefaces\scratchpads\glimmer\compat\core\src\main\resources\META-INF\resources\ice-extras.js
                Sending content: D:\work\development\head\svn\ossrepo\icefaces\scratchpads\glimmer\compat\core\src\main\resources\META-INF\resources\ice-extras.js
                Completed: At revision: 20256

                Show
                Adnan Durrani added a comment - fixed. Modified: D:\work\development\head\svn\ossrepo\icefaces\scratchpads\glimmer\compat\core\src\main\resources\META-INF\resources\ice-extras.js Sending content: D:\work\development\head\svn\ossrepo\icefaces\scratchpads\glimmer\compat\core\src\main\resources\META-INF\resources\ice-extras.js Completed: At revision: 20256
                Hide
                Permalink
                Adnan Durrani added a comment -

                The problem was related to an empty id of event target. I looked at the history and found out that the id was removed intentionally to fix the following JIRA:
                http://jira.icefaces.org/browse/ICE-2874

                In addition to that a null form was being passed to the iceSubmitPartial() call, where resetHiddenFieldsFor(form); was throwing an exception due to the null form.

                To fix this we are setting an id on the target element before submit and to preserve the fix of ICE-2874 we are resetting it back to empty string. As well as we are passing a valid form element to the iceSubmitPartial() call.

                Show
                Adnan Durrani added a comment - The problem was related to an empty id of event target. I looked at the history and found out that the id was removed intentionally to fix the following JIRA: http://jira.icefaces.org/browse/ICE-2874 In addition to that a null form was being passed to the iceSubmitPartial() call, where resetHiddenFieldsFor(form); was throwing an exception due to the null form. To fix this we are setting an id on the target element before submit and to preserve the fix of ICE-2874 we are resetting it back to empty string. As well as we are passing a valid form element to the iceSubmitPartial() call.
                Hide
                Permalink
                Mandeep Hayher added a comment -

                Tested successfully on ICEfaces 2.0.0-Alpha2 pre-release bundle using IE7 & IE6 browser.

                Show
                Mandeep Hayher added a comment - Tested successfully on ICEfaces 2.0.0-Alpha2 pre-release bundle using IE7 & IE6 browser.

                  People

                  • Assignee:
                    Mandeep Hayher
                    Reporter:
                    Adnan Durrani
                  • Votes:
                    0 Vote for this issue
                    Watchers:
                    0 Start watching this issue

                    Dates

                    • Created:
                      Updated:
                      Resolved: