Merge pull request #57 from carpentries/fix-sidebar-label

add border around navbar toggler
carpentries · Aug 11, 2022 · a9ce2cd · a9ce2cd
2 parents 06b7387 + eb64331
commit a9ce2cd
Show file tree

Hide file tree

Showing 4 changed files with 7 additions and 3 deletions.
diff --git a/NEWS.md b/NEWS.md
@@ -15,6 +15,11 @@
   #56)
 * The collapsed sidebar label no longer reverts to saying "collapse" on a new
   page. 
+* The hamburger menu for mobile devices now has a border so it's more clear
+  that it is a menu (suggested by @drmowinckels,
+  https://github.com/carpentries/workbench/issues/16 (#47); fixed by @zkamvar, 
+  #57)
+
 
 # varnish 0.2.3
 

diff --git a/inst/pkgdown/assets/assets/styles.css b/inst/pkgdown/assets/assets/styles.css
diff --git a/inst/pkgdown/assets/assets/styles.css.map b/inst/pkgdown/assets/assets/styles.css.map
diff --git a/source/stylesheets/header.scss b/source/stylesheets/header.scss
@@ -105,7 +105,6 @@
 
 .navbar-toggler {
   float: left;
-  border: none;
   color: $black;
   margin-left: 17px;
 }