Changes between Initial Version and Version 1 of Ticket #9087, comment 39


Ignore:
Timestamp:
Jun 5, 2015, 7:51:27 PM (6 years ago)
Author:
Kevin H. Kamel
Comment:

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #9087, comment 39

    initial v1  
    11Hi usmonster,
    22
    3 I agree with your assessment. The override for _moveToTop is no longer necessary. The initial fix from ticket 10269 appears to be the only thing we need for this to work in jQuery UI 1.11.4. I noticed your _allowInteraction code is different, I cannot confirm if that works or not. The variant included below has been in production for me for 18 months, and I know for sure that works.
     3I agree with your assessment. The override for _moveToTop is no longer necessary. The initial fix from ticket http://bugs.jqueryui.com/ticket/10269 appears to be the only thing we need for this to work in jQuery UI 1.11.4. I noticed your _allowInteraction code is different, I cannot confirm if that works or not. The variant included below has been in production for me for 18 months, and I know for sure that works.
    44
    55Fiddle: http://jsfiddle.net/kamelkev/80qex6v1/6/
    66
    7 Code that we have tested in IE 8/9/10, and latest versions of autoupdated browsers (Firefox,Chrome,Safari):
     7Code that we have tested in IE 9/10/11, and latest versions of autoupdated browsers (Firefox,Chrome,Safari):
    88
    99{{{