-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
[ui] Job status panel: tooltips on individual allocs #17514
[ui] Job status panel: tooltips on individual allocs #17514
Conversation
role="tooltip" | ||
aria-label="Allocations that have been rescheduled, on another node if possible, due to failure or manual restart" | ||
> | ||
<Hds::TooltipButton @text="Allocations that have been rescheduled, on another node if possible, due to failure or manual restart"> |
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.
Side-effect to update the tooltips here from custom-rolled ones to the official Helios tooltips.
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.
lgtm!
Ember Test Audit comparison
|
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.
Changes LGTM, but I ran into an error when running make dev-ui
:
Sass Syntax Error (SassCompiler) in /var/folders/sp/tb0_nysd40qf4w5jdhxgs8440000gp/T/broccoli-77574aZB57CvmdynR/out-700-funnel//app/styles/components/tooltip.scss:14:15
Error: Undefined variable.
╷
14 │ background: $black;
│ ^^^^^^
╵
/var/folders/sp/tb0_nysd40qf4w5jdhxgs8440000gp/T/broccoli-77574aZB57CvmdynR/out-700-funnel/app/styles/components/tooltip.scss 14:15 @use
/var/folders/sp/tb0_nysd40qf4w5jdhxgs8440000gp/T/broccoli-77574aZB57CvmdynR/out-700-funnel/app/styles/@hashicorp/design-system-components.scss 43:1 @import
/var/folders/sp/tb0_nysd40qf4w5jdhxgs8440000gp/T/broccoli-77574aZB57CvmdynR/out-700-funnel/app/styles/app.scss 11:9 root stylesheet
Stack Trace and Error Report: /var/folders/sp/tb0_nysd40qf4w5jdhxgs8440000gp/T/error.dump.30842f53940c5a14089c1686e757354d.log
make: *** [ember-dist] Error 1
I think our tooltip.scss
is conflicting with Helios tooltip.scss
.
I renamed our file to legacy-tooltip.scss
to fix it.
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.
All working now 🎉
Updates to the latest Helios (Hashicorp Design System) and includes new
<Hds::Tooltip>
elements to allocations in the status panel, showing theirshortId
.This does not add any tooltip to "grouped" allocation blocks (the "+4" and "+3" blocks in the screenshot below).