Fixes #545 text color of "about us" option and "news" option (#565)

* Fix dropdown structure: Remove unnecessary <li> tag for better responsiveness * Refactor: Change dropdown-menu tag from <ul> to <div>
sugarlabs · Dec 2, 2024 · 7dbca4a · 7dbca4a
1 parent 6c6ed31
commit 7dbca4a
Showing 1 changed file with 21 additions and 56 deletions.
diff --git a/_includes/nav.html b/_includes/nav.html
@@ -19,41 +19,18 @@
                   aria-expanded="false"
                   >About us<span class="caret"></span
                 ></a>
-                <ul class="dropdown-menu">
-                  <li>
-                    <a class="dropdown-item" href="{{ site.baseurl }}/about-us"
-                      >About us</a
-                    >
-                  </li>
-                  <li>
-                    <a
-                      class="dropdown-item"
-                      href="{{ site.baseurl }}/about-us/#mission"
-                      >Our mission</a
-                    >
-                  </li>
-                  <li>
-                    <a
-                      class="dropdown-item"
-                      href="{{ site.baseurl }}/leadership"
-                      >Leadership and governance</a
-                    >
-                  </li>
-                  <li>
-                    <a
-                      class="dropdown-item"
-                      href="{{ site.baseurl }}/contact-us"
-                      >Contact us</a
-                    >
-                  </li>
-                  <li>
-                    <a
-                      class="dropdown-item"
-                      href="{{ site.baseurl }}/FAQ"
-                      >FAQs</a
-                    >
-                  </li>
-                </ul>
+                <div class="dropdown-menu">
+                 <a class="dropdown-item" href="{{ site.baseurl }}/about-us"
+                 >About us</a>
+                 <a class="dropdown-item" href="{{ site.baseurl }}/about-us/#mission"
+                 >Our mission</a>
+                 <a class="dropdown-item" href="{{ site.baseurl }}/leadership"
+                 >Leadership and governance</a>
+                 <a class="dropdown-item" href="{{ site.baseurl }}/contact-us"
+                 >Contact us</a>
+                 <a class="dropdown-item" href="{{ site.baseurl }}/FAQ"
+                 >FAQs</a> 
+                </div>
               </li>
 
 
@@ -67,27 +44,15 @@
                   aria-expanded="false"
                   >News<span class="caret"></span
                 ></a>
-                <ul class="dropdown-menu">
-                  <li>
-                    <a class="dropdown-item" href="{{ site.baseurl }}/community-news"
-                      >Community News</a
-                    >
-                  </li>
-                  <li>
-                    <a
-                      class="dropdown-item"
-                      href="{{ site.baseurl }}/event"
-                      >Events</a
-                    >
-                  </li>
-                  <li>
-                    <a
-                      class="dropdown-item"
-                      href="{{ site.baseurl }}/press"
-                      >Press Release</a
-                    >
-                  </li>
-                </ul>
+                <div class="dropdown-menu">
+                <a class="dropdown-item"
+                  href="{{ site.baseurl }}/community-news"
+                  >Community News</a>
+                <a class="dropdown-item" href="{{ site.baseurl }}/event"
+                  >Events</a>
+                <a class="dropdown-item" href="{{ site.baseurl }}/press"
+                  >Press Release</a>
+                </div>
               </li>
 
               <li class="nav-item">