refactor: 💡 use hds side nav w/ hds frame instead of rose #2688
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
| {{/if}} | ||
| </Rose::Nav::Sidebar> | ||
| {{#if this.session.isAuthenticated}} | ||
| {{#in-element this.sideBarContainer}} |
There was a problem hiding this comment.
Why did we have to portal this in?
There was a problem hiding this comment.
thats a good point, i think the same thing can be accomplished using the side nav list directly.
There was a problem hiding this comment.
Edit: Just realized you are referring to the in-element helper usage. That is because we don't show the side nav on all routes but we do want the side nav to be inserted into the correct hds::frame component.
lisbet-alvarez
force-pushed
the
refactor-rose-side-bar-to-use-hds
branch
from
e30d653 to
0583534
Compare
| @@ -53,3 +53,20 @@ form { | |||
| color: inherit; | |||
| } | |||
| } | |||
|
|
|||
| // TODO: This is a temporary fix that ensures changing the color theme won't alter the dropdown colors. | |||
| .hds-app-frame__sidebar .hds-side-nav { | |||
There was a problem hiding this comment.
Hds team is helping us find a cleaner solution for this style override. So this fix is just temporary.
| @@ -37,7 +37,7 @@ | |||
| <page.actions> | |||
| {{#if (or perms.canUpdate perms.canDelete)}} | |||
|
|
|||
| <Hds::Dropdown as |dd|> | |||
| <Hds::Dropdown data-test-manage-alias as |dd|> | |||
There was a problem hiding this comment.
Do we want to specify that this is the manage alias dropdown in the targets route?
There was a problem hiding this comment.
ah great point!
| const DROPDOWN_BUTTON_SELECTOR = | ||
| 'tbody tr td:last-child .hds-dropdown-toggle-icon'; |
There was a problem hiding this comment.
asking for learning: Did this need to be updated since we have more dropdown (specifically in the new side nav) elements on the screen? Either way, I think this is a good change.
There was a problem hiding this comment.
yup that is correct!
Description
Refactor to use
Hds::SideNavandHds::AppFramecomponents.Note: There is an issue with the
Hds::SideNav::Listcomponent so we are using theHds::SideNav::Portalcomponent in the meantime.The side nav should not be visible on the
onboarding,change-password, andauthenticateroutes.Screenshots (if appropriate)
How to Test
a. user menu options such as sign-out, change-password, light mode, dark mode, etc.
b. dev tool toggle options such as feature editions and licensed features.
Checklist