Menu Items in leaf tree are not getting the correct underline decoration & pointers on hover,
due to css weight issue (id > class) probably due to the hardening of the navbar.module.css or missing the id.

.navbar-box a {
  cursor: pointer;
  text-decoration: none;
}
.navbar-box a:hover {
  text-decoration: underline;
}

is not applied

instead this style rule is applied

#navbar-administration *:not(select) {
  /* IE8 is the only browser we support that can't handle the :not
   * pseudo-selector. If users really want Navbar support in IE8 on sites with
   * themes that don't properly encapsulate their CSS, then they're going to
   * need to write that patch :) The reset is not strictly speaking, necessary.
   * It is provided as a convenience to protect again style bleed from themes
   * and modules that don't properly scope styling.
   *
   * The select element completely blows up if its properties are reset.
   */
  background-attachment: scroll;
  background-color: transparent;
  background-image: none;
  background-position: 0% 0%;
  background-repeat: no-repeat;
  border: none;
  -moz-box-shadow: none;
  box-shadow: none;
  -moz-box-sizing: border-box;
  -webkit-box-sizing: border-box;
  box-sizing: border-box;
  bottom: auto;
  color: black;
  content: normal;
  counter-increment: none;
  counter-reset: none;
  direction: ltr;
  float: none;
  font-style: normal;
  font-weight: normal;
  height: auto;
  left: auto;
  letter-spacing: normal;
  line-height: 1;
  list-style: none;
  margin: 0;
  max-height: none;
  max-width: none;
  min-height: 0;
  min-width: 0;
  orphans: 2;
  outline: none;
  outline-color: invert;
  outline-style: none;
  outline-width: medium;
  overflow: visible;
  padding: 0;
  page-break-after: auto;
  page-break-before: auto;
  page-break-inside: auto;
  position: static;
  right: auto;
  table-layout: auto;
  text-decoration: none;
  text-indent: 0;
  text-transform: none;
  top: auto;
  vertical-align: baseline;
  visibility: visible;
  widows: 2;
  z-index: auto;
  unicode-bidi: normal;
  white-space: normal;
  width: auto;
  word-spacing: normal;
}
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

rokat’s picture

Status: Needs review » Active
rokat’s picture

Provided potential fix for the issue, hope anyone could try it

rokat’s picture

Status: Active » Needs review
rokat’s picture

Issue summary: View changes
rokat’s picture

Priority: Normal » Minor
hass’s picture

rokat’s picture

Actually it's is not the case, i've tried in a theme less (empty style.css) installation.
The module is overriding it's own css.
The navbar.module.css is overriding navbar.theme.css , beacuse the css selector weight is less ,thus never applying the styles.

I can post images if needed.

hass’s picture

I guess we need to fix D8 first... Not sure.

tsvenson’s picture

hass’s picture

Status: Needs review » Closed (duplicate)
Parent issue: » #2119989: Add navbar_menu_tree() to prevent theme clashes