Skip to content

Commit 551fa98

Browse files
committed
Accordion: Moving IE layout fix (via zoom: 1) from js to css - fixes the destroy-method test in IEs (zoom style wasn't cleaned up on destroy) and should perform slightly faster, too
1 parent d5a6b33 commit 551fa98

File tree

2 files changed

+2
-5
lines changed

2 files changed

+2
-5
lines changed

themes/base/jquery.ui.accordion.css

+2
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,8 @@
44
.ui-accordion .ui-accordion-li-fix { display: inline; }
55
.ui-accordion .ui-accordion-header-active { border-bottom: 0 !important; }
66
.ui-accordion .ui-accordion-header a { display: block; font-size: 1em; padding: .5em .5em .5em .7em; }
7+
/* IE7-/Win - Fix extra vertical space in lists */
8+
.ui-accordion a { zoom: 1; }
79
.ui-accordion-icons .ui-accordion-header a { padding-left: 2.2em; }
810
.ui-accordion .ui-accordion-header .ui-icon { position: absolute; left: .5em; top: 50%; margin-top: -8px; }
911
.ui-accordion .ui-accordion-content { padding: 1em 2.2em; border-top: 0; margin-top: -2px; position: relative; top: 1px; margin-bottom: 2px; overflow: auto; display: none; zoom: 1; }

ui/jquery.ui.accordion.js

-5
Original file line numberDiff line numberDiff line change
@@ -74,11 +74,6 @@ $.widget("ui.accordion", {
7474
//Append icon elements
7575
this._createIcons();
7676

77-
// IE7-/Win - Extra vertical space in lists fixed
78-
if ($.browser.msie) {
79-
this.element.find('a').css('zoom', '1');
80-
}
81-
8277
this.resize();
8378

8479
//ARIA

0 commit comments

Comments
 (0)