-
Notifications
You must be signed in to change notification settings - Fork 31
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
feat: NavTabList component #1698
Conversation
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #1698 +/- ##
==========================================
- Coverage 46.68% 46.45% -0.23%
==========================================
Files 609 611 +2
Lines 36985 37198 +213
Branches 9306 9357 +51
==========================================
+ Hits 17265 17279 +14
- Misses 19668 19865 +197
- Partials 52 54 +2
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Functionality looks good, context menu should be generated on click though
953f7fc
to
ecd8a57
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, that's the correct dh-color variable.
This is ported from DHE and generalized a bit so it can be used in DHC and DHE. Also cleaned up some of the CSS, but could still use some improvements for theming.
Test in the styleguide Navigations section. The 1st tab in both lists is non-closable. Add custom context actions for the styleguide which selects the tab to the left/right of the tab you opened the context menu on.