Permalink
Browse files

Menu: Removed space character prefixes from summary elements in small…

… view.

#7942 modified the mobile menu plugin to generate span elements within summary elements. But as a part of that change, the plugin began inserting a space character at the start of each summary. This negatively impacted their visual layouts, since the extra space is unnecessary and since it causes summaries to look horizontally misaligned (compared to nearby links).

This commit fixes the layout issues by removing the aforementioned space character from the menu plugin.
  • Loading branch information...
EricDunsworth committed Jul 5, 2017
1 parent 5943530 commit 1dc786b585b3dc61250972b0f6246dfb20e0d7bd
Showing with 1 addition and 1 deletion.
  1. +1 −1 src/plugins/menu/menu.js
@@ -116,7 +116,7 @@ var componentName = "wb-menu",
menuitem = " role='menuitem'",
sectionHtml = "<li><details>" + "<summary class='mb-item" +
( $section.hasClass( "wb-navcurr" ) || $section.children( ".wb-navcurr" ).length !== 0 ? " wb-navcurr'" : "'" ) +
" aria-haspopup='true'> <span" + menuitem + ">" +
" aria-haspopup='true'><span" + menuitem + ">" +
$section.text() + "</span></summary>" +
"<ul class='list-unstyled mb-sm' role='menu' aria-expanded='false' aria-hidden='true'>";

0 comments on commit 1dc786b

Please sign in to comment.