Theme: removed or changed occurrences of zoom. Fixes #7975 - Remove invalid CSS for legacy browsers

selfthinker · mikesherov · commit d7bff010691b · 2012-11-29T21:48:58.000-05:00
Because of overlapping issues, this also reverts most of e77edc6 and fixes it in a different way.
diff --git a/demos/button/toolbar.html b/demos/button/toolbar.html
@@ -11,12 +11,12 @@
 	<link rel="stylesheet" href="../demos.css">
 	<style>
 	#toolbar {
-		padding: 11px 4px 9px 4px;
+		padding: 4px;
+		display: inline-block;
 	}
-
 	/* support: IE7 */
-	*:first-child+html #toolbar {
-		padding: 4px 0px 4px 5px;
+	*+html #toolbar {
+		display: inline;
 	}
 	</style>
 	<script>
@@ -91,7 +91,7 @@
 </head>
 <body>
 
-<span id="toolbar" class="ui-widget-header ui-corner-all">
+<div id="toolbar" class="ui-widget-header ui-corner-all">
 	<button id="beginning">go to beginning</button>
 	<button id="rewind">rewind</button>
 	<button id="play">play</button>
@@ -106,12 +106,12 @@
 		<input type="radio" id="repeat1" name="repeat" /><label for="repeat1">Once</label>
 		<input type="radio" id="repeatall" name="repeat" /><label for="repeatall">All</label>
 	</span>
-</span>
+</div>
 
 <div class="demo-description">
 <p>
 	A mediaplayer toolbar. Take a look at the underlying markup: A few button elements,
-	an input of type checkbox for the Shuffle button, and three inputs of type radio for the Repeat options. 
+	an input of type checkbox for the Shuffle button, and three inputs of type radio for the Repeat options.
 </p>
 </div>
 </body>
diff --git a/themes/base/jquery.ui.accordion.css b/themes/base/jquery.ui.accordion.css
@@ -14,7 +14,7 @@
 	position: relative;
 	margin-top: 2px;
 	padding: .5em .5em .5em .7em;
-	zoom: 1;
+	min-height: 0; /* support: IE7 */
 }
 .ui-accordion .ui-accordion-icons {
 	padding-left: 2.2em;
@@ -35,5 +35,4 @@
 	padding: 1em 2.2em;
 	border-top: 0;
 	overflow: auto;
-	zoom: 1;
 }
diff --git a/themes/base/jquery.ui.button.css b/themes/base/jquery.ui.button.css
@@ -17,7 +17,6 @@
 	cursor: pointer;
 	vertical-align: middle;
 	text-align: center;
-	zoom: 1;
 	overflow: visible; /* removes extra width in IE */
 }
 .ui-button,
@@ -44,7 +43,7 @@ button.ui-button-icons-only {
 
 /* button text element */
 .ui-button .ui-button-text {
-	display: inline-block;
+	display: block;
 	line-height: normal;
 }
 .ui-button-text-only .ui-button-text {
diff --git a/themes/base/jquery.ui.core.css b/themes/base/jquery.ui.core.css
@@ -43,7 +43,7 @@
 	clear: both;
 }
 .ui-helper-clearfix {
-	zoom: 1;
+	min-height: 0; /* support: IE7 */
 }
 .ui-helper-zfix {
 	width: 100%;
diff --git a/themes/base/jquery.ui.dialog.css b/themes/base/jquery.ui.dialog.css
@@ -40,7 +40,6 @@
 	padding: .5em 1em;
 	background: none;
 	overflow: auto;
-	zoom: 1;
 }
 .ui-dialog .ui-dialog-buttonpane {
 	text-align: left;
diff --git a/themes/base/jquery.ui.menu.css b/themes/base/jquery.ui.menu.css
@@ -22,7 +22,6 @@
 .ui-menu .ui-menu-item {
 	margin: 0;
 	padding: 0;
-	zoom: 1;
 	width: 100%;
 }
 .ui-menu .ui-menu-divider {
@@ -37,7 +36,7 @@
 	display: block;
 	padding: 2px .4em;
 	line-height: 1.5;
-	zoom: 1;
+	min-height: 0; /* support: IE7 */
 	font-weight: normal;
 }
 .ui-menu .ui-menu-item a.ui-state-focus,
diff --git a/themes/base/jquery.ui.tabs.css b/themes/base/jquery.ui.tabs.css
@@ -11,7 +11,6 @@
 .ui-tabs {
 	position: relative;/* position: relative prevents IE scroll bug (element with position: relative inside container with overflow: auto appear as "fixed") */
 	padding: .2em;
-	zoom: 1;
 }
 .ui-tabs .ui-tabs-nav {
 	margin: 0;
diff --git a/themes/base/jquery.ui.theme.css b/themes/base/jquery.ui.theme.css
@@ -41,9 +41,10 @@
 	background: #cccccc/*{bgColorHeader}*/ url(images/ui-bg_highlight-soft_75_cccccc_1x100.png)/*{bgImgUrlHeader}*/ 50%/*{bgHeaderXPos}*/ 50%/*{bgHeaderYPos}*/ repeat-x/*{bgHeaderRepeat}*/;
 	color: #222222/*{fcHeader}*/;
 	font-weight: bold;
-	zoom: 1;
 }
-.ui-widget-header a { color: #222222/*{fcHeader}*/; }
+.ui-widget-header a {
+	color: #222222/*{fcHeader}*/;
+}
 
 /* Interaction states
 ----------------------------------*/

Original file line number	Diff line number	Diff line change
`@@ -14,7 +14,7 @@`
`14`	`14`	`position: relative;`
`15`	`15`	`margin-top: 2px;`
`16`	`16`	`padding: .5em .5em .5em .7em;`
`17`		`- zoom: 1;`
	`17`	`+ min-height: 0; /* support: IE7 */`
`18`	`18`	`}`
`19`	`19`	`.ui-accordion .ui-accordion-icons {`
`20`	`20`	`padding-left: 2.2em;`
`@@ -35,5 +35,4 @@`
`35`	`35`	`padding: 1em 2.2em;`
`36`	`36`	`border-top: 0;`
`37`	`37`	`overflow: auto;`
`38`		`- zoom: 1;`
`39`	`38`	`}`
Original file line number	Diff line number	Diff line change
`@@ -43,7 +43,7 @@`
`43`	`43`	`clear: both;`
`44`	`44`	`}`
`45`	`45`	`.ui-helper-clearfix {`
`46`		`- zoom: 1;`
	`46`	`+ min-height: 0; /* support: IE7 */`
`47`	`47`	`}`
`48`	`48`	`.ui-helper-zfix {`
`49`	`49`	`width: 100%;`
Original file line number	Diff line number	Diff line change
`@@ -40,7 +40,6 @@`
`40`	`40`	`padding: .5em 1em;`
`41`	`41`	`background: none;`
`42`	`42`	`overflow: auto;`
`43`		`- zoom: 1;`
`44`	`43`	`}`
`45`	`44`	`.ui-dialog .ui-dialog-buttonpane {`
`46`	`45`	`text-align: left;`
Original file line number	Diff line number	Diff line change
`@@ -11,7 +11,6 @@`
`11`	`11`	`.ui-tabs {`
`12`	`12`	`position: relative;/* position: relative prevents IE scroll bug (element with position: relative inside container with overflow: auto appear as "fixed") */`
`13`	`13`	`padding: .2em;`
`14`		`- zoom: 1;`
`15`	`14`	`}`
`16`	`15`	`.ui-tabs .ui-tabs-nav {`
`17`	`16`	`margin: 0;`