From 551fa98ae4e1eafd3a152ca4897166537a069f16 Mon Sep 17 00:00:00 2001 From: jzaefferer Date: Sat, 27 Mar 2010 08:03:48 +0100 Subject: [PATCH] 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 --- themes/base/jquery.ui.accordion.css | 2 ++ ui/jquery.ui.accordion.js | 5 ----- 2 files changed, 2 insertions(+), 5 deletions(-) diff --git a/themes/base/jquery.ui.accordion.css b/themes/base/jquery.ui.accordion.css index acd7f5e1e..8d8a1a6e1 100644 --- a/themes/base/jquery.ui.accordion.css +++ b/themes/base/jquery.ui.accordion.css @@ -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; } diff --git a/ui/jquery.ui.accordion.js b/ui/jquery.ui.accordion.js index d1a7abd6d..bce78231b 100644 --- a/ui/jquery.ui.accordion.js +++ b/ui/jquery.ui.accordion.js @@ -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