Skip to content
Permalink
Browse files

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
  • Loading branch information
jzaefferer committed Mar 27, 2010
1 parent d5a6b33 commit 551fa98ae4e1eafd3a152ca4897166537a069f16
Showing with 2 additions and 5 deletions.
  1. +2 −0 themes/base/jquery.ui.accordion.css
  2. +0 −5 ui/jquery.ui.accordion.js
@@ -4,6 +4,8 @@
.ui-accordion .ui-accordion-li-fix { display: inline; }
.ui-accordion .ui-accordion-header-active { border-bottom: 0 !important; }
.ui-accordion .ui-accordion-header a { display: block; font-size: 1em; padding: .5em .5em .5em .7em; }
/* IE7-/Win - Fix extra vertical space in lists */
.ui-accordion a { zoom: 1; }
.ui-accordion-icons .ui-accordion-header a { padding-left: 2.2em; }
.ui-accordion .ui-accordion-header .ui-icon { position: absolute; left: .5em; top: 50%; margin-top: -8px; }
.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; }
@@ -74,11 +74,6 @@ $.widget("ui.accordion", {
//Append icon elements
this._createIcons();

// IE7-/Win - Extra vertical space in lists fixed
if ($.browser.msie) {
this.element.find('a').css('zoom', '1');
}

this.resize();

//ARIA

0 comments on commit 551fa98

Please sign in to comment.
You can’t perform that action at this time.