Skip to content

Commit

Permalink
fix: Ask component's Home button causes header height unconformity vs…
Browse files Browse the repository at this point in the history
… ProfileExplorer
  • Loading branch information
starpit committed Nov 13, 2022
1 parent 86bb5ad commit fdf9b5e
Show file tree
Hide file tree
Showing 3 changed files with 23 additions and 2 deletions.
4 changes: 2 additions & 2 deletions plugins/plugin-codeflare/src/components/Ask.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -109,9 +109,9 @@ export default class AskUI extends React.PureComponent<Props, State> {
/** content to show in the upper right */
private actions() {
return (
<ActionGroup>
<ActionGroup className="kui--guide-actions">
<Tooltip markdown={`### Home\n#### Jump back to the beginning\n\n⌘ or Alt-click to open a new window`}>
<Button variant="link" icon={<HomeIcon className="larger-text" />} onClick={this._home} />
<Button variant="link" icon={<HomeIcon />} onClick={this._home} />
</Tooltip>
</ActionGroup>
)
Expand Down
15 changes: 15 additions & 0 deletions plugins/plugin-codeflare/web/scss/components/Ask.scss
Original file line number Diff line number Diff line change
Expand Up @@ -46,3 +46,18 @@
}
}
}

@include Ask {
.pf-c-card__actions {
align-self: unset;
}
@include AskActions {
button {
padding-top: 0;
padding-bottom: 0;
svg {
font-size: 1.25rem;
}
}
}
}
6 changes: 6 additions & 0 deletions plugins/plugin-codeflare/web/scss/components/_mixins.scss
Original file line number Diff line number Diff line change
Expand Up @@ -19,3 +19,9 @@
@content;
}
}

@mixin AskActions {
.kui--guide-actions {
@content;
}
}

0 comments on commit fdf9b5e

Please sign in to comment.