From 6d7589351eae1e180579fc0c87885e5ab7a73f48 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Scott=20Gonz=C3=A1lez?= <scott.gonzalez@gmail.com>
Date: Mon, 20 Oct 2014 19:18:34 -0400
Subject: [PATCH] All: Drop support for IE7

Fixes #9838
Closes gh-1365
---
 demos/button/toolbar.html                  | 4 ----
 tests/unit/accordion/accordion.html        | 4 ----
 tests/unit/dialog/dialog_methods.js        | 3 ++-
 tests/unit/dialog/dialog_options.js        | 8 ++++----
 tests/unit/menu/menu.html                  | 4 ----
 tests/unit/selectable/selectable_events.js | 7 ++-----
 themes/base/accordion.css                  | 1 -
 themes/base/core.css                       | 3 ---
 themes/base/menu.css                       | 1 -
 themes/base/selectmenu.css                 | 2 --
 ui/tabs.js                                 | 4 ----
 11 files changed, 8 insertions(+), 33 deletions(-)
diff --git a/demos/button/toolbar.html b/demos/button/toolbar.html
index 4a93c5ccf..e2eac055c 100644
--- a/demos/button/toolbar.html
+++ b/demos/button/toolbar.html
@@ -14,10 +14,6 @@
 		padding: 4px;
 		display: inline-block;
 	}
-	/* support: IE7 */
-	*+html #toolbar {
-		display: inline;
-	}
 	</style>
 	<script>
 	$(function() {
diff --git a/tests/unit/accordion/accordion.html b/tests/unit/accordion/accordion.html
index 5a76ba566..0a8755fd3 100644
--- a/tests/unit/accordion/accordion.html
+++ b/tests/unit/accordion/accordion.html
@@ -35,10 +35,6 @@
 		font-size: 12px;
 		line-height: 15px;
 	}
-	/* avoid IE7 oscillating between overflow visible and scroll values */
-	#list1 > div {
-		overflow: visible;
-	}
 	</style>
 </head>
 <body>
diff --git a/tests/unit/dialog/dialog_methods.js b/tests/unit/dialog/dialog_methods.js
index d315e5fc3..c449ad0d4 100644
--- a/tests/unit/dialog/dialog_methods.js
+++ b/tests/unit/dialog/dialog_methods.js
@@ -193,7 +193,8 @@ test("open", function() {
 	ok(element.dialog("widget").is(":visible") && !element.dialog("widget").is(":hidden"), "dialog visible after open method called");
 });
 
-test("#6137: dialog('open') causes form elements to reset on IE7", function() {
+// http://bugs.jqueryui.com/ticket/6137
+test("Ensure form elements don't reset when opening a dialog", function() {
 	expect(2);
 
 	var d1 = $("<form><input type='radio' name='radio' id='a' value='a' checked='checked'></input>" +
diff --git a/tests/unit/dialog/dialog_options.js b/tests/unit/dialog/dialog_options.js
index 87f3553be..3e5444c67 100644
--- a/tests/unit/dialog/dialog_options.js
+++ b/tests/unit/dialog/dialog_options.js
@@ -371,9 +371,9 @@ test("minWidth", function() {
 test( "position, default center on window", function() {
 	expect( 2 );
 
-	// dialogs alter the window width and height in FF and IE7
+	// dialogs alter the window width and height in Firefox
 	// so we collect that information before creating the dialog
-	// Support: FF, IE7
+	// Support: Firefox
 	var winWidth = $( window ).width(),
 		winHeight = $( window ).height(),
 		element = $("<div></div>").dialog(),
@@ -387,9 +387,9 @@ test( "position, default center on window", function() {
 test( "position, right bottom at right bottom via ui.position args", function() {
 	expect( 2 );
 
-	// dialogs alter the window width and height in FF and IE7
+	// dialogs alter the window width and height in Firefox
 	// so we collect that information before creating the dialog
-	// Support: FF, IE7
+	// Support: Firefox
 	var winWidth = $( window ).width(),
 		winHeight = $( window ).height(),
 		element = $("<div></div>").dialog({
diff --git a/tests/unit/menu/menu.html b/tests/unit/menu/menu.html
index 5eda3f41e..cb3df5b00 100644
--- a/tests/unit/menu/menu.html
+++ b/tests/unit/menu/menu.html
@@ -35,10 +35,6 @@
 	}
 	.ui-menu .ui-menu-item {
 		padding: 0;
-		/* Support: IE7 */
-		*display: block;
-		*float: left;
-		*clear: left;
 	}
 	#menu3 {
 		height: 250px;
diff --git a/tests/unit/selectable/selectable_events.js b/tests/unit/selectable/selectable_events.js
index 2cb99f49c..5df7f2f5b 100644
--- a/tests/unit/selectable/selectable_events.js
+++ b/tests/unit/selectable/selectable_events.js
@@ -49,12 +49,9 @@ test( "mousedown: initial position of helper", function() {
 		clientY: 10
 	});
 
-	// we do a GTE comparison here because IE7 erroneously subtracts
-	// 2 pixels from a simulated mousedown for clientX/Y
-	// Support: IE7
 	helperOffset = $( ".ui-selectable-helper" ).offset();
-	ok( helperOffset.top >= 99, "Scroll top should be accounted for." );
-	ok( helperOffset.left >= 99, "Scroll left should be accounted for." );
+	ok( helperOffset.top, 110, "Scroll top should be accounted for." );
+	ok( helperOffset.left, 110, "Scroll left should be accounted for." );
 
 	// Cleanup
 	element.simulate( "mouseup" );
diff --git a/themes/base/accordion.css b/themes/base/accordion.css
index 43560be03..10d5ff82e 100644
--- a/themes/base/accordion.css
+++ b/themes/base/accordion.css
@@ -14,7 +14,6 @@
 	position: relative;
 	margin: 2px 0 0 0;
 	padding: .5em .5em .5em .7em;
-	min-height: 0; /* support: IE7 */
 	font-size: 100%;
 }
 .ui-accordion .ui-accordion-icons {
diff --git a/themes/base/core.css b/themes/base/core.css
index e66fcf32a..b1831c8f8 100644
--- a/themes/base/core.css
+++ b/themes/base/core.css
@@ -43,9 +43,6 @@
 .ui-helper-clearfix:after {
 	clear: both;
 }
-.ui-helper-clearfix {
-	min-height: 0; /* support: IE7 */
-}
 .ui-helper-zfix {
 	width: 100%;
 	height: 100%;
diff --git a/themes/base/menu.css b/themes/base/menu.css
index 2be35870f..58c561349 100644
--- a/themes/base/menu.css
+++ b/themes/base/menu.css
@@ -23,7 +23,6 @@
 	margin: 0;
 	padding: 3px 1em 3px .4em;
 	cursor: pointer;
-	min-height: 0; /* support: IE7 */
 	/* support: IE10, see #8844 */
 	list-style-image: url("data:image/gif;base64,R0lGODlhAQABAIAAAAAAAP///yH5BAEAAAAALAAAAAABAAEAAAIBRAA7");
 }
diff --git a/themes/base/selectmenu.css b/themes/base/selectmenu.css
index b362956a6..93c74e00d 100644
--- a/themes/base/selectmenu.css
+++ b/themes/base/selectmenu.css
@@ -18,8 +18,6 @@
 }
 .ui-selectmenu-menu .ui-menu {
 	overflow: auto;
-	/* Support: IE7 */
-	overflow-x: hidden;
 	padding-bottom: 1px;
 }
 .ui-selectmenu-menu .ui-menu .ui-selectmenu-optgroup {
diff --git a/ui/tabs.js b/ui/tabs.js
index e45a97c72..fea7138d5 100644
--- a/ui/tabs.js
+++ b/ui/tabs.js
@@ -48,10 +48,6 @@ return $.widget( "ui.tabs", {
 		return function( anchor ) {
 			var anchorUrl, locationUrl;
 
-			// support: IE7
-			// IE7 doesn't normalize the href property when set via script (#9317)
-			anchor = anchor.cloneNode( false );
-
 			anchorUrl = anchor.href.replace( rhash, "" );
 			locationUrl = location.href.replace( rhash, "" );
 
