Resizable: Apply zIndex option to all handles. Partial Fix for #7960-…

… Modal dialog does not disable resizables on the page.
jquery · Apr 20, 2012 · 6150abae49f77a3b2f67c39a3c60a06792b4de8f · 6150aba
1 parent d783766
commit 6150abae49f77a3b2f67c39a3c60a06792b4de8f
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 4 deletions.
diff --git a/tests/unit/resizable/resizable_options.js b/tests/unit/resizable/resizable_options.js
@@ -187,4 +187,13 @@ test("ui-resizable-nw { handles: 'all', minWidth: 60, minHeight: 60, maxWidth: 1
 	equal( 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);
diff --git a/themes/base/jquery.ui.resizable.css b/themes/base/jquery.ui.resizable.css
@@ -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; }

diff --git a/ui/jquery.ui.resizable.js b/ui/jquery.ui.resizable.js
@@ -99,9 +99,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) {