Skip to content

Commit

Permalink
[Security Solution] Visualisation actions button is overlap on to the…
Browse files Browse the repository at this point in the history
… bar chart (elastic#145973)

## Summary

Original issue: Visualisation actions button is overlap on to the bar
chart.
elastic#128344

Before:
<img width="575" alt="Screenshot 2022-11-22 at 11 31 19"
src="https://user-images.githubusercontent.com/6295984/203304147-ff0ec823-182c-40bf-8da5-fa45180eae0a.png">

After:
<img width="765" alt="Screenshot 2022-11-22 at 11 30 39"
src="https://user-images.githubusercontent.com/6295984/203304150-5dae9a4a-341d-4717-abb1-f56aa20ba6bd.png">
  • Loading branch information
angorayc authored Nov 23, 2022
1 parent e00e26e commit a065866
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ import {
WrappedByAutoSizer,
useTheme,
Wrapper,
ChartWrapper,
BarChartWrapper,
} from './common';
import { DraggableLegend } from './draggable_legend';
import type { LegendItem } from './draggable_legend_item';
Expand Down Expand Up @@ -209,7 +209,7 @@ export const BarChartComponent: React.FC<BarChartComponentProps> = ({
<Wrapper>
<HoverVisibilityContainer targetClassNames={[HISTOGRAM_ACTIONS_BUTTON_CLASS]}>
{isValidSeriesExist && barChart && (
<ChartWrapper gutterSize="none">
<BarChartWrapper gutterSize="none">
<EuiFlexItem grow={true}>
<WrappedByAutoSizer ref={measureRef} height={chartHeight}>
<BarChartBase
Expand All @@ -226,7 +226,7 @@ export const BarChartComponent: React.FC<BarChartComponentProps> = ({
<LegendFlexItem grow={false}>
<DraggableLegend legendItems={legendItems} height={height} />
</LegendFlexItem>
</ChartWrapper>
</BarChartWrapper>
)}
{!isValidSeriesExist && (
<ChartPlaceHolder height={chartHeight} width={chartWidth} data={barChart} />
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -154,3 +154,8 @@ export const Wrapper = styled.div`
export const ChartWrapper = styled(EuiFlexGroup)`
z-index: 0;
`;

export const BarChartWrapper = styled(EuiFlexGroup)`
z-index: 0;
padding-right: 20px;
`;

0 comments on commit a065866

Please sign in to comment.