From 606ab8c643ebdb90d3ba4e48fe253a1ed7ae2c0d Mon Sep 17 00:00:00 2001
From: sbertal <sbertal@yahoo-inc.com>
Date: Fri, 23 Dec 2016 06:11:53 -0800
Subject: [PATCH] reset style for horizontal dropdown separator (fixes #545)

---
 src/menus/css/menus-skin.css      |  9 ++++++++-
 src/menus/tests/manual/menus.html | 28 +++++++++++++++++++++++++++-
 2 files changed, 35 insertions(+), 2 deletions(-)

diff --git a/src/menus/css/menus-skin.css b/src/menus/css/menus-skin.css
index 94dd254..52cc621 100644
--- a/src/menus/css/menus-skin.css
+++ b/src/menus/css/menus-skin.css
@@ -1,6 +1,7 @@
 /* misc default styling */
 
-.pure-menu-separator {
+.pure-menu-separator,
+.pure-menu-horizontal .pure-menu-children .pure-menu-separator {
     background-color: #ccc;
     height: 1px;
     margin: .3em 0;
@@ -12,6 +13,12 @@
     margin: 0 .3em ;
 }
 
+/* Need to reset the separator since submenu is vertical */
+.pure-menu-horizontal .pure-menu-children .pure-menu-separator {
+    display: block;
+    width: auto;
+}
+
 .pure-menu-heading {
     text-transform: uppercase;
     color: #565d64;
diff --git a/src/menus/tests/manual/menus.html b/src/menus/tests/manual/menus.html
index 5acd82c..9df8e4c 100644
--- a/src/menus/tests/manual/menus.html
+++ b/src/menus/tests/manual/menus.html
@@ -10,6 +10,7 @@
     <script src="http://yui.yahooapis.com/3.17.2/build/yui/yui-min.js"></script>
     <style>
         h1 {
+            margin-top: 2em;
             font-size: 15px;
             font-weight: normal;
         }
@@ -194,6 +195,7 @@
                         <a href="#" class="pure-menu-link">More Stuff</a>
                         <ul class="pure-menu-children">
                             <li class="pure-menu-item"><a class="pure-menu-link" href="/handlebars">Foo</a></li>
+                            <li class="pure-menu-separator"></li>
                             <li class="pure-menu-item"><a class="pure-menu-link" href="/dust">Bar</a></li>
                             <li class="pure-menu-item"><a class="pure-menu-link" href="/react">Baz</a></li>
                         </ul>
@@ -206,6 +208,30 @@
         </ul>
     </div>
 
+    <h1>div.pure-menu.pure-menu-horizontal with nested dropdown and separator</h1>
+    <div class="pure-menu pure-menu-horizontal">
+        <a href="#" class="pure-menu-link pure-menu-heading">Title</a>
+        <ul class="pure-menu-list">
+            <li class="pure-menu-item"><a href="#" class="pure-menu-link">Home</a></li>
+            <li class="pure-menu-item"><a href="#" class="pure-menu-link">About</a></li>
+            <li class="pure-menu-item"><a href="#" class="pure-menu-link">Blog</a></li>
+            <li class="pure-menu-item"><a href="#" class="pure-menu-link">Contact</a></li>
+            <li class="pure-menu-item"><a href="#" class="pure-menu-link">GitHub</a></li>
+            <li class="pure-menu-item"><a href="#" class="pure-menu-link">Twitter</a></li>
+            <li class="pure-menu-item"><a href="#" class="pure-menu-link">Wang</a></li>
+            <li class="pure-menu-item pure-menu-has-children pure-menu-allow-hover">
+                <a href="#" class="pure-menu-link">Yahoo</a>
+                <ul class="pure-menu-children">
+                    <li class="pure-menu-item"><a class="pure-menu-link" href="/handlebars">Foo</a></li>
+                    <li class="pure-menu-separator"></li>
+                    <li class="pure-menu-item"><a class="pure-menu-link" href="/dust">Bar</a></li>
+                    <li class="pure-menu-item"><a class="pure-menu-link" href="/react">Baz</a></li>
+                </ul>
+            </li>
+        </ul>
+    </div>
+
+
     <h1>div.pure-menu.pure-menu-horizontal.pure-menu-scrollable</h1>
     <div class="pure-menu pure-menu-horizontal pure-menu-scrollable">
         <a href="#" class="pure-menu-link pure-menu-heading">Title</a>
@@ -262,7 +288,7 @@
             <li class="pure-menu-item"><a href="#" class="link-custom pure-menu-link">Twitter</a></li>
         </ul>
     </div>
-    <script src="http://10.73.144.35:5000/js/menus.js"></script>
+    <!-- script src="http://10.73.144.35:5000/js/menus.js"></script -->
 
 </body>
 </html>