Two sets of changes:
-
Updates the navbar docs to put
.activeon.nav-linkinstead of.nav-itemlike the rest of our navs. Fixes #30652 (closed). -
Removes two selectors,
.active > .nav-linkand.nav-link.show, because as far as I can tell, those are duplicative and unused, respectively..activejust needs to be on the.nav-link, and.showis always on the parent.nav-itembecause it toggles the.dropdown-menu.
Am I missing something with those two selectors? Clicking around on dropdowns in the nav shows no issues thus far.
Regarding consistency, I also just noticed that Punted to #30846..btns get .btn.show for the dropdown-menu though. Do we consolidate and make sure .show is always on the .btn and .nav-link instead of parents?