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

Hotfix: GH-157 Frontera Redesign Use Global Color Vars #210

Merged

Conversation

wesleyboar
Copy link
Member

@wesleyboar wesleyboar commented Apr 27, 2021

Overview

Fix gray text in header on Frontera User Guide.

I.e. fix bug introduced by #181.

Changes

  1. Fix: Include global color variables (only needed for User Guide).*

* This solution can be removed in PR #192.

Testing

  1. On https://dev.fronteraweb.tacc.utexas.edu/user-guide/, ensure header text is white, not gray.

Screenshots

Before

Pre-Prod (Before)

Pre-Prod Frontera Docs sans Fix

Pre-Prod Frontera Docs sans Fix

Pre-Prod Frontera Portal sans Fix

Pre-Prod Frontera Portal sans Fix

Pre-Prod Frontera CMS sans Fix

Pre-Prod Frontera CMS sans Fix

Local (Before)

Local Frontera Docs sans Fix

Local Frontera Docs sans Fix

Local Frontera Portal sans Fix

Local Frontera Portal sans Fix

Local Frontera CMS sans Fix

Local Frontera CMS sans Fix

After

Dev (After)

Dev Frontera Docs with Fix

Dev Frontera Docs with Fix

Dev Frontera Portal with Fix

Dev Frontera Portal with Fix

Dev Frontera CMS with Fix

Dev Frontera CMS with Fix

Local (After)

Local Frontera Docs with Fix

Local Frontera Docs with Fix

Local Frontera Portal with Fix

Local Frontera Portal with Fix

Local Frontera CMS with Fix

Local Frontera CMS with Fix

Notes

The search bar is bigger on Portal than in CMS and User Guide. This is true on production, pre-prod, dev, and local. It will be fixed separately from this hotfix.

@wesleyboar wesleyboar changed the title GH-157: Fix PR #181 Gray Portal Nav Text for Docs hotfix/gh 157 frontera redesign gh 181 use global color vars Apr 27, 2021
@wesleyboar wesleyboar requested review from rstijerina and taoteg April 27, 2021 20:59
Comment on lines +13 to +15
/* NOTE: Portal does not need this, because it has independent copy of these,
but the User Guides do not. This can be removed in Core-CMS PR #192 */
@import url("../../../site_shared/css/src/_imports/settings/color.css");
Copy link
Member Author

@wesleyboar wesleyboar Apr 27, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Portal and CMS both having independent copies of global color variables is a problem I'll solve later this year.

@wesleyboar wesleyboar merged commit b3757c9 into main Apr 28, 2021
@wesleyboar wesleyboar deleted the hotfix/GH-157-frontera-redesign--GH-181-use-global-color-vars branch April 28, 2021 21:22
@wesleyboar wesleyboar linked an issue Jun 30, 2021 that may be closed by this pull request
@wesleyboar wesleyboar changed the title hotfix/gh 157 frontera redesign gh 181 use global color vars Hotfix: GH-157 Frontera Redesign, GH-181 Use Global Color Vars Sep 3, 2021
@wesleyboar wesleyboar changed the title Hotfix: GH-157 Frontera Redesign, GH-181 Use Global Color Vars Hotfix: GH-157 Frontera Redesign Use Global Color Vars Sep 3, 2021
@wesleyboar wesleyboar added this to the Frontera Redesign Phase 0 milestone Sep 3, 2021
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.

2 participants