Merge pull request #1 from sheadawson/master

buxfix menu to display nicely in ie7-8
symbiote · May 29, 2012 · d1bf745 · d1bf745
2 parents c620bc0 + 4de8bda
commit d1bf745
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 3 deletions.
diff --git a/css/GroupedCmsMenu.css b/css/GroupedCmsMenu.css
@@ -1,6 +1,7 @@
 #top #MainMenu li.children > a {
 	background: url(../images/bg-menu-children.gif) no-repeat top right;
 	padding-right: 17px;
+	float: none;
 }
 
 #top #MainMenu li.current.children > a {
@@ -25,9 +26,10 @@
 
 #top #MainMenu li ul {
 	display: none;
+	float:none;
 	position: absolute;
 	z-index: 1000;
-	margin-top: 33px;
+	top: 33px;
 	border: 1px solid #AAA;
 	border-top: none;
 	margin-left: -1px;
@@ -37,7 +39,8 @@
 }
 
 #top #MainMenu ul li {
-	display: inline;
+	display: block;
+	width: 100%;
 	float: none;
 	margin: 0;
 }
@@ -46,7 +49,6 @@
 	background: #6A7580;
 	display: block;
 	float: none;
-	padding-right: 30px;
 }
 
 #top #MainMenu li.current ul a {

diff --git a/javascript/GroupedCmsMenu.js b/javascript/GroupedCmsMenu.js
@@ -3,4 +3,10 @@
 		function() { $(this).addClass("active").find("ul").show(); },
 		function() { $(this).removeClass("active").find("ul").hide(); }
 	);
+
+	// ie hack to set css width explicitly on sub menus
+	// so that the sub menu items can expand to that width
+	$('#MainMenu li.children ul').each(function(){
+		$(this).css('width', $(this).width());
+	});
 })(jQuery);