]> git.ipfire.org Git - thirdparty/bootstrap.git/commit
Update nav and navbar HTML markup
authorGeremia Taglialatela <tagliala.dev@gmail.com>
Wed, 12 Apr 2017 14:06:52 +0000 (16:06 +0200)
committerMark Otto <markd.otto@gmail.com>
Sat, 27 May 2017 22:21:49 +0000 (15:21 -0700)
commitdb44e4b311317ef760f8412cc33c84146959b248
treedaf89d9f3d96fab2276ff368ec6969c5681a494e
parentf2829dd6ea10d1af7e451397f7228087cff95b8a
Update nav and navbar HTML markup

According to the docs, you just need to apply the `navbar-nav` to an
`<ul>` tag in a list-based navbar. In fact, `navbar-nav` seems to
override all the css properties set by `nav`.

Also, `nav-item` class should be used only for `<li>` tags, or in
conjunction with `<a>` tags to entirely avoid the list-based approach.

This commit removes the redundant `nav` class from `navbar-nav` `<ul>`
tags, moves `nav-item` from `<a>` tags to `<li>` tags and properly
adds `nav-item` class to `<li>` tags where missing.

Refs:

* https://v4-alpha.getbootstrap.com/components/navs/
* https://v4-alpha.getbootstrap.com/components/navbar/#nav
docs/_includes/nav-home.html
js/tests/unit/scrollspy.js
js/tests/visual/modal.html