Merge lp:~quam-plures-core/quam-plures/qp-validate_toolbar into lp:quam-plures

[Yabs]

Validated toolbar

[EdB (edb)]

Tested and found to be awesome. Will merge immediately given that waiting for any other core team members to do their code review is not a good idea.

review: Approve

=== modified file 'inc/_core/ui/_menu.class.php'
--- inc/_core/ui/_menu.class.php	2009-12-19 19:51:52 +0000
+++ inc/_core/ui/_menu.class.php	2010-03-11 09:45:27 +0000
@@ -332,7 +332,7 @@
 					'afterEach' => '</li>',
 					'beforeEachSel' => '<li class="current">',
 					'afterEachSel' => '</li>',
-					'separator' => '<li class="separator"><a><hr /></a></li>',
+					'separator' => '<li class="separator"><a href="#"><span>&nbsp;</span></a></li>',
 					'disabled_class' => 'disabled',
 					'_props' => array(
 						'recurse' => 'always',  // options are: 'no' 'always' or 'intoselected'

=== modified file 'rsc/css/basic.css'
--- rsc/css/basic.css	2010-03-06 16:19:45 +0000
+++ rsc/css/basic.css	2010-03-11 09:45:27 +0000
@@ -240,7 +240,7 @@
 .sf-menu li.separator a:hover {
 	background-color: ThreeDFace;
 }
-.sf-menu hr {
+.sf-menu .separator span {
 	height: 0px;
 	display: block;
 	padding: 0;
@@ -250,7 +250,7 @@
 	border-bottom: 1px solid ThreeDHighlight;
 	width: 100%;
 }
-.toolbar_msie hr {
+.toolbar_msie .separator span {
 	position: relative;
 	top: -6px;
 	margin: 6px 0 0;

=== modified file 'templates/_toolbar.inc.php'
--- templates/_toolbar.inc.php	2010-03-06 16:19:45 +0000
+++ templates/_toolbar.inc.php	2010-03-11 09:45:27 +0000
@@ -79,7 +79,6 @@
 	}
 }
 ?>
-</ul>
 </div>
 
 </div>