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 May 14, 2021
- Load new styles solely for docs header.
- Force branding styles on old markup.
- Migrate subsite selectors from `trumps/s-header` to `tools/selectors`.
- Rename `:--in-__subsite__` selectors to `:--from-__subsite__`.
- Create new styles solely for docs header.
- Document "For (Other Sites)" i.e. `trumps/for-docs.*.css` stylesheets.
wesleyboar added a commit that referenced this issue May 14, 2021
Unset Docs' `img { max-width; }`. Docs does this too but for old markup.
wesleyboar added a commit that referenced this issue May 14, 2021
- Install `merge-lite` for deep merge constants and theme data.
- Use env() vars for header height.
- Remove unnecessary code and improve comments.
- Create constants for theme JSON.
- Merge constants on top of theme data.
- Update theme data comment syntax.
wesleyboar added a commit that referenced this issue Aug 18, 2021
CAVEAT: Font is not always consistent between states and sites.
wesleyboar added a commit that referenced this issue Aug 19, 2021
CAVEAT: The mobile nav menus have not yet been moved ot their new position in the design.
wesleyboar added a commit that referenced this issue Aug 19, 2021
CAVEAT: The mobile nav menus have not yet been moved ot their new position in the design.
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