Skip to content
This repository

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP
Browse code

accordion: fix for #4436

  • Loading branch information...
commit e132c1b363e66b0a4a03eeb17c4fdbf316c982fd 1 parent 5c5e083
Jörn Zaefferer authored August 11, 2009
50  tests/visual/accordion/accordion_option_fillSpace_true.html
... ...
@@ -0,0 +1,50 @@
  1
+<!doctype html>
  2
+<html lang="en">
  3
+<head>
  4
+	<title>Accordion Visual Test : Accordion option autoHeight true</title>
  5
+	<link rel="stylesheet" href="../visual.css" type="text/css" />
  6
+	<link rel="stylesheet" href="../../../themes/base/ui.all.css" type="text/css">
  7
+	<script type="text/javascript" src="../../../jquery-1.3.2.js"></script>
  8
+	<script type="text/javascript" src="../../../ui/ui.core.js"></script>
  9
+	<script type="text/javascript" src="../../../ui/ui.accordion.js"></script>
  10
+	<script type="text/javascript">
  11
+	$(function() {
  12
+		$("#accordion").accordion({
  13
+			fillSpace: true
  14
+		});
  15
+	})
  16
+	</script>
  17
+</head>
  18
+<body>
  19
+
  20
+<div style="height: 500px; width: 500px; border: 1px solid red;">
  21
+	<div id="accordion" style="width:490px;">
  22
+		<h3><a href="#">Accordion Header 1</a></h3>
  23
+		<div>
  24
+			Accordion Content 1
  25
+		</div>
  26
+		<h3><a href="#">Accordion Header 2</a></h3>
  27
+		<div>
  28
+			Accordion Content 2
  29
+			<p>paragraph</p>
  30
+			<p>paragraph</p>
  31
+			<p>paragraph</p>
  32
+			<p>paragraph</p>
  33
+			<p>paragraph</p>
  34
+			<p>paragraph</p>
  35
+			<p>paragraph</p>
  36
+		</div>
  37
+		<h3><a href="#">Accordion Header 3</a></h3>
  38
+		<div>
  39
+			Accordion Content 3
  40
+			<ul>
  41
+				<li>list item</li>
  42
+				<li>list item</li>
  43
+				<li>list item</li>
  44
+			</ul>
  45
+		</div>
  46
+	</div>
  47
+</div>
  48
+
  49
+</body>
  50
+</html>
2  ui/ui.accordion.js
@@ -213,7 +213,7 @@ $.widget("ui.accordion", {
213 213
 			if($.browser.msie) { this.element.parent().css('overflow', defOverflow); }
214 214
 	
215 215
 			this.headers.each(function() {
216  
-				maxHeight -= $(this).outerHeight();
  216
+				maxHeight -= $(this).outerHeight(true);
217 217
 			});
218 218
 
219 219
 			var maxPadding = 0;

0 notes on commit e132c1b

Please sign in to comment.
Something went wrong with that request. Please try again.