Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

refactor(tup-cms): FAVICON, LOGO, TACC_PORTAL_ #416

Open
wants to merge 12 commits into
base: main
Choose a base branch
from

Conversation

wesleyboar
Copy link
Member

@wesleyboar wesleyboar commented Jan 31, 2024

Overview

Support TACC/Core-CMS rename of some settings.

Related

Changes

  • refactor use of CMS settings that have changed

Testing

  1. Verify logo markup is unchanged.
  2. Verify favicon markup is unchanged.
  3. Verify news list layout is unchanged.
  4. Verify news article header layout is unchanged.

UI

Branding (Before) Branding (After)
Logo
Favicon
News (Before) News (After)
List
Article Header

@wesleyboar wesleyboar changed the title chore(tup-cms): prepend PORTAL_ to FAVICON & LOGO chore(tup-cms): FAVICON, LOGO, TACC_PORTAL Jan 31, 2024
@wesleyboar wesleyboar changed the title chore(tup-cms): FAVICON, LOGO, TACC_PORTAL refactor(tup-cms): FAVICON, LOGO, TACC_PORTAL Jan 31, 2024
@wesleyboar wesleyboar changed the title refactor(tup-cms): FAVICON, LOGO, TACC_PORTAL refactor(tup-cms): FAVICON, LOGO, TACC_PORTAL_ Jan 31, 2024
wesleyboar and others added 5 commits March 6, 2024 16:46
With this update, there is an unexpected and bewildering UI change:
- the navbar at 992px is collapsed
- it should only collapse at 991px
- this behavior is different than when using Core-CMS v4.8.3
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.

None yet

1 participant