Skip to content

fix: s-header nav-link alignment inconsistent#427

Merged
wesleyboar merged 1 commit intomainfrom
fix/from-core-cms-custom-409
Jan 17, 2025
Merged

fix: s-header nav-link alignment inconsistent#427
wesleyboar merged 1 commit intomainfrom
fix/from-core-cms-custom-409

Conversation

@wesleyboar
Copy link
Copy Markdown
Member

@wesleyboar wesleyboar commented Jan 15, 2025

Overview

Fix vertical alignment of .nav-links in CMS menu to work regardless of navbar height and width.

Inconsistency can occur 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; }.

Related

Changes

  • fixes & refactors header nav link styles

Testing

  1. Replace the content of a site's core-styles.header.css with new dist.
    1. https://dev.cep.tacc.utexas.edu
    2. https://pprd.lccf-portal.tacc.utexas.edu
      fix(lccf): align cms menu nav link via Core-Styles Core-CMS-Custom#410
  2. View when navbar is expanded.
    1. Verify .nav-link underline is at bottom of header.
    2. Verify .nav-link text is vertically aligned center with search bar.
  3. View when navbar is collapsed.
    1. Verify .nav-links are not too close to each other.

UI

LCCF Custom header.css Core-Styles …header.css
lccf custom header css content core-styles header css content
lccf custom header css url core-styles header css url
mobile view
mobile view

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant