-
Notifications
You must be signed in to change notification settings - Fork 1
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
Labels
effort XL
Extra large effort task (~4 weeks) — does not include review time
enhancement
Improvements or additions to existing features
priority ▲
High priority
Milestone
Comments
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
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.
This was referenced May 12, 2021
wesleyboar
added a commit
that referenced
this issue
May 12, 2021
This was
linked to
pull requests
May 13, 2021
Merged
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
wesleyboar
added a commit
that referenced
this issue
May 14, 2021
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 17, 2021
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
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
Sep 2, 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 10, 2021
wesleyboar
added a commit
that referenced
this issue
Sep 23, 2021
wesleyboar
added a commit
that referenced
this issue
Sep 27, 2021
wesleyboar
added a commit
that referenced
this issue
Nov 4, 2021
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
Labels
effort XL
Extra large effort task (~4 weeks) — does not include review time
enhancement
Improvements or additions to existing features
priority ▲
High priority
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
(Confluence) Shared UI - Components - Header (see "Mid 2021")
(Adobe XD) CMS-Common Components
If you need custom navbar transisitons instead of Bootstrap's…
The text was updated successfully, but these errors were encountered: