Skip to content

Commit

Permalink
Merge pull request #3389 from relative-ci/fix-run-info-icon
Browse files Browse the repository at this point in the history
fix(ui): RunInfo - hide tooltip help icon by default
  • Loading branch information
vio authored May 22, 2023
2 parents b055f5b + e2cf85a commit 2815ab9
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 4 deletions.
6 changes: 4 additions & 2 deletions packages/ui/src/components/run-info/run-info.module.css
Original file line number Diff line number Diff line change
Expand Up @@ -40,13 +40,15 @@
}

/* Hover state */
.titleIcon span[aria-controls=""] {
.titleHoverCardIcon span[aria-controls=""],
.titleTooltipIcon[aria-controls=""] {
opacity: 0;
visibility: none;
transition: var(--ui-transition-out);
}

.root:hover .titleIcon span {
.root:hover .titleHoverCardIcon span,
.root:hover .titleTooltipIcon {
opacity: 1;
visibility: visible;
transition: var(--ui-transition-in);
Expand Down
7 changes: 5 additions & 2 deletions packages/ui/src/components/run-info/run-info.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -73,12 +73,15 @@ export const RunInfo = ({
<FlexStack space="xxxsmall" alignItems="center" as="h3" className={css.title}>
<span>{title}</span>
{titleHoverCard && (
<HoverCard label={<Icon glyph={Icon.ICONS.HELP} />} className={css.titleIcon}>
<HoverCard
label={<Icon glyph={Icon.ICONS.HELP} />}
className={cx(css.titleIcon, css.titleHoverCardIcon)}
>
{titleHoverCard}
</HoverCard>
)}
{titleTooltip && (
<Tooltip title={titleTooltip} className={css.titleIcon}>
<Tooltip title={titleTooltip} className={cx(css.titleIcon, css.titleTooltipIcon)}>
<Icon glyph={Icon.ICONS.HELP} />
</Tooltip>
)}
Expand Down

0 comments on commit 2815ab9

Please sign in to comment.