r3046 committed - accordion: fix for #4436
Revision: 3046
Author: joern.zaefferer
Date: Tue Aug 11 11:46:00 2009
Log: accordion: fix for #4436
http://code.google.com/p/jquery-ui/source/detail?r=3046
Added:
/trunk/tests/visual/accordion/accordion_option_fillSpace_true.html
Modified:
/trunk/ui/ui.accordion.js
=======================================
--- /dev/null
+++ /trunk/tests/visual/accordion/accordion_option_fillSpace_true.html Tue
Aug 11 11:46:00 2009
@@ -0,0 +1,50 @@
+<!doctype html>
+<html lang="en">
+<head>
+ <title>Accordion Visual Test : Accordion option autoHeight true</title>
+ <link rel="stylesheet" href="../visual.css" type="text/css" />
+ <link rel="stylesheet" href="../../../themes/base/ui.all.css"
type="text/css">
+ <script type="text/javascript" src="../../../jquery-1.3.2.js"></script>
+ <script type="text/javascript" src="../../../ui/ui.core.js"></script>
+ <script type="text/javascript" src="../../../ui/ui.accordion.js"></script>
+ <script type="text/javascript">
+ $(function() {
+ $("#accordion").accordion({
+ fillSpace: true
+ });
+ })
+ </script>
+</head>
+<body>
+
+<div style="height: 500px; width: 500px; border: 1px solid red;">
+ <div id="accordion" style="width:490px;">
+ <h3><a href="#">Accordion Header 1</a></h3>
+ <div>
+ Accordion Content 1
+ </div>
+ <h3><a href="#">Accordion Header 2</a></h3>
+ <div>
+ Accordion Content 2
+ paragraph
+ paragraph
+ paragraph
+ paragraph
+ paragraph
+ paragraph
+ paragraph
+ </div>
+ <h3><a href="#">Accordion Header 3</a></h3>
+ <div>
+ Accordion Content 3
+ <ul>
+ <li>list item</li>
+ <li>list item</li>
+ <li>list item</li>
+ </ul>
+ </div>
+ </div>
+</div>
+
+</body>
+</html>
=======================================
--- /trunk/ui/ui.accordion.js Mon Aug 10 17:45:01 2009
+++ /trunk/ui/ui.accordion.js Tue Aug 11 11:46:00 2009
@@ -213,7 +213,7 @@
if($.browser.msie) { this.element.parent().css('overflow',
defOverflow); }
this.headers.each(function() {
- maxHeight -= $(this).outerHeight();
+ maxHeight -= $(this).outerHeight(true);
});
var maxPadding = 0;