Skip to content

Commit

Permalink
Resizable: Apply zIndex option to all handles. Partial Fix for #7960-…
Browse files Browse the repository at this point in the history
… Modal dialog does not disable resizables on the page.

(cherry picked from commit 6150aba)
  • Loading branch information
tjvantoll authored and scottgonzalez committed Apr 20, 2012
1 parent 849b562 commit 8ce8b77
Show file tree
Hide file tree
Showing 3 changed files with 12 additions and 4 deletions.
9 changes: 9 additions & 0 deletions tests/unit/resizable/resizable_options.js
Expand Up @@ -187,4 +187,13 @@ test("ui-resizable-nw { handles: 'all', minWidth: 60, minHeight: 60, maxWidth: 1
equals( target.height(), 100, "compare maxHeight" );
});

test("zIndex, applied to all handles", function() {
expect(8);

var target = $('<div></div>').resizable({ handles: 'all', zIndex: 100 });
target.children( '.ui-resizable-handle' ).each( function( index, handle ) {
equals( $( handle ).css( 'zIndex' ), 100, 'compare zIndex' );
});
});

})(jQuery);
2 changes: 1 addition & 1 deletion themes/base/jquery.ui.resizable.css
Expand Up @@ -8,7 +8,7 @@
* http://docs.jquery.com/UI/Resizable#theming
*/
.ui-resizable { position: relative;}
.ui-resizable-handle { position: absolute;font-size: 0.1px;z-index: 99999; display: block; }
.ui-resizable-handle { position: absolute;font-size: 0.1px; display: block; }
.ui-resizable-disabled .ui-resizable-handle, .ui-resizable-autohide .ui-resizable-handle { display: none; }
.ui-resizable-n { cursor: n-resize; height: 7px; width: 100%; top: -5px; left: 0; }
.ui-resizable-s { cursor: s-resize; height: 7px; width: 100%; bottom: -5px; left: 0; }
Expand Down
5 changes: 2 additions & 3 deletions ui/jquery.ui.resizable.js
Expand Up @@ -98,9 +98,8 @@ $.widget("ui.resizable", $.ui.mouse, {
var handle = $.trim(n[i]), hname = 'ui-resizable-'+handle;
var axis = $('<div class="ui-resizable-handle ' + hname + '"></div>');

// increase zIndex of sw, se, ne, nw axis
//TODO : this modifies original option
if(/sw|se|ne|nw/.test(handle)) axis.css({ zIndex: ++o.zIndex });
// Apply zIndex to all handles - see #7960
axis.css({ zIndex: o.zIndex });

//TODO : What's going on here?
if ('se' == handle) {
Expand Down

0 comments on commit 8ce8b77

Please sign in to comment.