This started out as a general cleanup of our pseudo-classes (to simplify our CSS output), but quickly turned into a smorgasbord of commits. At a high level, this affects navs, navbars, pagination, and list groups, but here's a small rundown of the highlights:
-
Removes
plain-hover-focusmixins from navs, navbar navs, pagination, and list groups. As noted in #21439, these things often don't need to be reset for the active/disabled states. This really helps to reduce the compiled CSS selectors. -
Overhauls the list group a bit to provide better docs, improve Sass variable indentation, cleans up some styles for lighter output.
-
Updates pagination to use flexbox instead of floats and what not. Fixes #20029 (closed) in the process.
-
Revamps pagination docs to include simpler examples, incorporate accessibility notes into the core of the docs instead of as callouts, and provide examples of using flex alignment.
-
Fixes some instances of
.openwhich was recently changed to.showfor toggling dropdowns.
My biggest question in this is, do still need these hover mixins for this stuff? @cvrebertc, hoping we can perhaps delete the plain-hover-focus one since that should be shied away from anyway.