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

Header Redesign → FP-1458 #101

Closed
wesleyboar opened this issue Jan 29, 2021 · 1 comment · Fixed by #220, #226, #227, #228 or #239
Closed

Header Redesign → FP-1458 #101

wesleyboar opened this issue Jan 29, 2021 · 1 comment · Fixed by #220, #226, #227, #228 or #239
Labels
effort XL Extra large effort task (~4 weeks) — does not include review time enhancement Improvements or additions to existing features priority ▲ High priority

Comments

@wesleyboar
Copy link
Member

wesleyboar commented Jan 29, 2021

Status

Migrated to FP-1458.

Goal

Direction

Unless requested, avoid performing technical debt and nice to haves, like:

The styles are not ideal, but there are maintainable. The markup is duplicated, but the duplication is much less than it was before you took over.

Reference

@wesleyboar wesleyboar added enhancement Improvements or additions to existing features priority ▼ Low priority effort S Small effort task (~½ a week) — does not include review time tacc labels Jan 29, 2021
@wesleyboar wesleyboar added priority ▲ High priority and removed priority ▼ Low priority labels Apr 22, 2021
wesleyboar added a commit that referenced this issue May 11, 2021
- Migrate `branding_logos` CSS to `c-branding` & `c-logo`.
- In `s-header` style `.c-logo` not `.navbar-brand` .
- Update and format markup for `header_branding` and `header_logo`.
- Comment on now-unused secrets values.
wesleyboar added a commit that referenced this issue May 11, 2021
- Migrate `branding_logos` CSS to `c-branding` & `c-logo`.
- In `s-header` style `.c-logo` not `.navbar-brand`.
- Update and format markup for `header_branding` and `header_logo`.
- Comment on now-unused secrets values.
wesleyboar added a commit that referenced this issue May 13, 2021
wesleyboar added a commit that referenced this issue May 13, 2021
This was linked to pull requests May 13, 2021
wesleyboar added a commit that referenced this issue May 13, 2021
- Remove now-moot taccsite_custom:frontera-cms search bar font sizes.
- Support CSS custom selectors (known external bug).
- Test CSS custom selectors (known external bug).
- Remove outdated comment from `c-branding`.
- Set `c-logo` image height in component stylesheet.
- Add TODO for `html { font-size: 62.5%; }`
- Add arrow "mixin" (note: using `%-arrow` syntax).
- Redesign header (but not dropdowns and not portal nav).
- Remove now-moot `s-portal-nav` styles.
- Add & Update header env variables for themes.
- Perform "GH-129: Do Not Support CMS Menu Nav Link & Dropdown".
wesleyboar added a commit that referenced this issue May 14, 2021
Prevent dropdown link from being flagged as active.
wesleyboar added a commit that referenced this issue May 14, 2021
- (Real Fix) Ensure colors are available to Docs.
- (Possible Fix) Ensure correct colors are available to Portal.\*

\* The Portal has its own settings/color.css that is almost identical.
wesleyboar added a commit that referenced this issue Aug 20, 2021
wesleyboar added a commit that referenced this issue Aug 23, 2021
wesleyboar added a commit that referenced this issue Sep 2, 2021
I am suprised more branches aren't failing, given that this caused a server crash on task/GH-101.
wesleyboar added a commit that referenced this issue Sep 2, 2021
wesleyboar added a commit that referenced this issue Sep 9, 2021
Known Issues:
1. On CMS & Docs, `s-header__menu-backdrop` adds gap in navigation bar.
2. On CMS, mobile navbar opens and closes too slow.
3. login link is a bit too wide
wesleyboar added a commit that referenced this issue Sep 9, 2021
Known Issues:
1. On CMS & Docs, `s-header__menu-backdrop` adds gap in navigation bar.
2. On CMS, mobile navbar opens and closes too slow.
3. The login link is a bit too wide.
4. Code is difficult to follow (but documented).
wesleyboar added a commit that referenced this issue Sep 23, 2021
@wesleyboar wesleyboar changed the title Header Redesign Header Redesign → FP-1458 Jan 12, 2022
@wesleyboar
Copy link
Member Author

Migrated to JIRA as FP-1458 a long time ago.

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