Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Won't Fix
    • Affects Version/s: 2.0-Beta2
    • Fix Version/s: 2.0.0
    • Component/s: None
    • Labels:
      None
    • Environment:
      Icefaces2 # 22965
      Server: Tomcat6
      Browser: IE8

      Description

      IE8 browser
      Animation: fade animation doesnot look right
      Date Time Entry: same problem with Render as popup & Render an input field
      TabSet (client side): orientation applied after tabbing out

      This problem is common on all browsers:
      TabSet (server side): selectTab3 and close it, now select tab two and try closing it ( tab two can not be closed)

        Activity

        Mandeep Hayher created issue -
        Mandeep Hayher made changes -
        Field Original Value New Value
        Assignee Ken Fyten [ ken.fyten ]
        Ken Fyten made changes -
        Salesforce Case []
        Fix Version/s 2.0.0 [ 10230 ]
        Assignee Priority P1
        Assignee Ken Fyten [ ken.fyten ] Adnan Durrani [ adnan.durrani ]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #22986 Sun Oct 31 00:18:04 MDT 2010 adnan.durrani Fix towards ICE-6175 (Tabset common problem with facet close button)
        Files Changed
        Commit graph MODIFY /icefaces2/trunk/icefaces/ace/component/src/org/icefaces/component/tab/TabSetRenderer.java
        Adnan Durrani made changes -
        Status Open [ 1 ] In Progress [ 3 ]
        Hide
        Adnan Durrani added a comment -

        Tabset common problem fixed related to facet close.

        command: Commit
        Modified: D:\work\development\head\svn\ossrepo\icefaces2\trunk\icefaces\ace\component\src\org\icefaces\component\tab\TabSetRenderer.java
        Sending content: D:\work\development\head\svn\ossrepo\icefaces2\trunk\icefaces\ace\component\src\org\icefaces\component\tab\TabSetRenderer.java
        Completed: At revision: 22986

        Show
        Adnan Durrani added a comment - Tabset common problem fixed related to facet close. command: Commit Modified: D:\work\development\head\svn\ossrepo\icefaces2\trunk\icefaces\ace\component\src\org\icefaces\component\tab\TabSetRenderer.java Sending content: D:\work\development\head\svn\ossrepo\icefaces2\trunk\icefaces\ace\component\src\org\icefaces\component\tab\TabSetRenderer.java Completed: At revision: 22986
        Hide
        Adnan Durrani added a comment -

        It came across that the issue is with the singleSubmit component. This component takes different path under IE8. Please see the attached snapshot of singleSubmit related js.

        We do submit on "change" and "blur" event with all browsers and under IE we call submit on "focusout"event. So that is why under IE we don't see any change on checkbox selection until we tab out. So the fix needs to go in singleSubmit JS. I tried to use some other techniques, but under IE I am getting "event bubbling" behavior not "event capturing".

        By looking at the code, I assume that we want to use event CAPTURING not event BUBBLING:
        http://www.quirksmode.org/js/events_order.html

        In past I have noticed there were some utility js, which normalizes this call for all browsers. Mircea might know about it.

        Show
        Adnan Durrani added a comment - It came across that the issue is with the singleSubmit component. This component takes different path under IE8. Please see the attached snapshot of singleSubmit related js. We do submit on "change" and "blur" event with all browsers and under IE we call submit on "focusout"event. So that is why under IE we don't see any change on checkbox selection until we tab out. So the fix needs to go in singleSubmit JS. I tried to use some other techniques, but under IE I am getting "event bubbling" behavior not "event capturing". By looking at the code, I assume that we want to use event CAPTURING not event BUBBLING: http://www.quirksmode.org/js/events_order.html In past I have noticed there were some utility js, which normalizes this call for all browsers. Mircea might know about it.
        Adnan Durrani made changes -
        Attachment ICE-6175.jpg [ 12663 ]
        Hide
        Ken Fyten added a comment -

        On 07/11/10 6:08 PM, Ted Goddard wrote:

        singleSubmit uses bubbling for efficiency – otherwise we would need to add
        markup throughout the entire form. Unfortunately, I believe that "blur"
        does not bubble on IE.

        Ted.


        On 08/11/10 9:14 AM, Mircea Toma wrote:

        See this document:
        http://www.quirksmode.org/blog/archives/2008/04/delegating%5Fthe.html .

        It seems that onfocusin and onfocusout could be used.

        Show
        Ken Fyten added a comment - On 07/11/10 6:08 PM, Ted Goddard wrote: singleSubmit uses bubbling for efficiency – otherwise we would need to add markup throughout the entire form. Unfortunately, I believe that "blur" does not bubble on IE. Ted. On 08/11/10 9:14 AM, Mircea Toma wrote: See this document: http://www.quirksmode.org/blog/archives/2008/04/delegating%5Fthe.html . It seems that onfocusin and onfocusout could be used.
        Hide
        Ken Fyten added a comment -

        Assign to Mircea to complete as this is related to ice:singleSubmit functionality.

        Show
        Ken Fyten added a comment - Assign to Mircea to complete as this is related to ice:singleSubmit functionality.
        Ken Fyten made changes -
        Salesforce Case []
        Assignee Priority P1 P2
        Assignee Adnan Durrani [ adnan.durrani ] Mircea Toma [ mircea.toma ]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #23113 Sat Nov 13 13:07:42 MST 2010 mircea.toma ICE-6175 Reformat code.
        Files Changed
        Commit graph MODIFY /icefaces2/trunk/icefaces/core/src/main/javascript/application.js
        Hide
        Mircea Toma added a comment -

        'onchange' event cannot be used effectively in IE because the event is fired only when the modified element loses the focus. See http://msdn.microsoft.com/en-us/library/ms536912(VS.85).aspx . So, even if the event would propagate (bubble or capture) it still won't be fired as in the other browsers.

        Show
        Mircea Toma added a comment - 'onchange' event cannot be used effectively in IE because the event is fired only when the modified element loses the focus. See http://msdn.microsoft.com/en-us/library/ms536912(VS.85).aspx . So, even if the event would propagate (bubble or capture) it still won't be fired as in the other browsers.
        Hide
        Mircea Toma added a comment -

        This document http://www.quirksmode.org/blog/archives/2008/04/delegating_the.html explains why you need to register the event handlers for 'onblur' and 'onfocus' for the capturing phase, it is only during this phase these events propagate through the DOM tree. So, the current code is correct since is registering the event handlers (onblur and onfocus) for the capturing phase.

        Show
        Mircea Toma added a comment - This document http://www.quirksmode.org/blog/archives/2008/04/delegating_the.html explains why you need to register the event handlers for 'onblur' and 'onfocus' for the capturing phase, it is only during this phase these events propagate through the DOM tree. So, the current code is correct since is registering the event handlers (onblur and onfocus) for the capturing phase.
        Hide
        Mircea Toma added a comment -

        Close this as "Won't fix" since there is no viable solution for fixing the 'onchange' behavior in IE. Also the current code is correct in using capturing phase for 'onblur' and 'onfocus'.

        Show
        Mircea Toma added a comment - Close this as "Won't fix" since there is no viable solution for fixing the 'onchange' behavior in IE. Also the current code is correct in using capturing phase for 'onblur' and 'onfocus'.
        Mircea Toma made changes -
        Status In Progress [ 3 ] Resolved [ 5 ]
        Resolution Won't Fix [ 2 ]
        Ken Fyten made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Assignee Priority P2

          People

          • Assignee:
            Mircea Toma
            Reporter:
            Mandeep Hayher
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: