fix(lccf): cms menu .nav-link alignment #409
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Overview
For LCCF, fix vertical alignment of
.nav-links
in CMS menu.This is a problem on all sites using
TACC_CORE_STYLES_VERSION = 2
but is most-noticeable on LCCF, because LCCF sets.s-header.navbar { --nav-padding-vert: 10px; }
(and.s-header .nav-link { top: unset; }
to partially resolve alignment.)Warning
Introduces nav link height bug on "mobile" navbar, but will be fixed by TACC/Core-Styles#427.
Related
Changes
.s-header .nav-link
styles(in a way that should be applied to Core-Styles instead)
Testing & UI
LCCF Custom Styles
header.css
with the new content..nav-link
underline is at bottom of header..nav-link
text is vertically aligned center with search bar.apply.updated.header.css.to.lccf.mov
Default CMS & Styles
/* To fix nav link alignment regardless the nav padding */
.nav-link
underline is at bottom of header..nav-link
text is vertically aligned center with search bar. larger */`.apply.core-styles.fix.to.cep.mov