Skip to content

Commit

Permalink
Merge pull request #12577 from influxdata/fix/veo-height
Browse files Browse the repository at this point in the history
Fix/veo height
  • Loading branch information
bthesorceror authored Mar 12, 2019
2 parents 05c591b + 7af5e4a commit ab2fd3b
Show file tree
Hide file tree
Showing 3 changed files with 14 additions and 4 deletions.
8 changes: 6 additions & 2 deletions ui/src/clockface/components/overlays/Overlay.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,7 @@ import {ErrorHandling} from 'src/shared/decorators/errors'
interface Props {
children: JSX.Element
visible: boolean
className?: string
}

interface State {
Expand Down Expand Up @@ -88,9 +89,12 @@ class Overlay extends Component<Props, State> {
}

private get overlayClass(): string {
const {visible} = this.props
const {visible, className} = this.props

return classnames('overlay', {show: visible})
return classnames('overlay', {
show: visible,
[`${className}`]: className,
})
}

private hideChildren = (): void => {
Expand Down
2 changes: 1 addition & 1 deletion ui/src/dashboards/components/DashboardPage.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -202,7 +202,7 @@ class DashboardPage extends Component<Props, State> {
onAddCell={this.handleAddCell}
/>
)}
<Overlay visible={isShowingVEO}>
<Overlay visible={isShowingVEO} className="veo-overlay">
<VEO onHide={this.handleHideVEO} onSave={this.handleSaveVEO} />
</Overlay>
</HoverTimeProvider>
Expand Down
8 changes: 7 additions & 1 deletion ui/src/dashboards/components/VEO.scss
Original file line number Diff line number Diff line change
Expand Up @@ -25,6 +25,12 @@
display: flex;
align-items: stretch;
width: 100%;
flex: 0 0 calc(100% - #{$page-header-size});
flex: 0 0 calc(100% - #{$page-header-size});
padding-bottom: $ix-marg-c;
}

.veo-overlay {
.overlay--transition {
height: 100%;
}
}

0 comments on commit ab2fd3b

Please sign in to comment.