Skip to content

Commit

Permalink
fix(FilterSidePanel/VerticalTabs): Adjust top and bottom spacing of f…
Browse files Browse the repository at this point in the history
…ilter-side-panel and vertical-t (#877)

.filter-panel-pf has top and bottom padding, which causes layout issues when it's the only item
being used.  Removing the top and bottom padding and adding a bottom margin fixes the bug and
provides the proper spacing if another element is used.  As a result of these changes,
.vertical-tabs-pf needs to have a larger bottom margin set on it.
  • Loading branch information
rhamilto authored and dtaylor113 committed Nov 7, 2018
1 parent 11aff34 commit abc8cc1
Show file tree
Hide file tree
Showing 5 changed files with 18 additions and 6 deletions.
Original file line number Diff line number Diff line change
@@ -1,5 +1,6 @@
.filter-panel-pf {
padding: 20px 15px;
margin: 0 0 30px;
padding: 0 15px;
}

.filter-panel-pf-category {
Expand Down
Original file line number Diff line number Diff line change
@@ -1,6 +1,11 @@
.vertical-tabs-pf {
padding: 0;
list-style: none;
margin: 0 0 30px;
padding: 0;

.vertical-tabs-pf {
margin-bottom: 0;
}
}

.vertical-tabs-pf-tab {
Expand Down
Original file line number Diff line number Diff line change
@@ -1,5 +1,6 @@
.filter-panel-pf {
padding: 20px 15px;
margin: 0 0 30px;
padding: 0 15px;
}

.filter-panel-pf-category {
Expand Down
Original file line number Diff line number Diff line change
@@ -1,6 +1,11 @@
.vertical-tabs-pf {
padding: 0;
list-style: none;
margin: 0 0 30px;
padding: 0;

.vertical-tabs-pf {
margin-bottom: 0;
}
}

.vertical-tabs-pf-tab {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ const stories = storiesOf(
stories.addDecorator(
defaultTemplate({
title: 'Filter Side Panel',
description: 'Note: the width and border styling is not part of the FilterSidePanel.'
description: 'Note: the width, border, and top padding styling are not part of the FilterSidePanel.'
})
);

Expand Down Expand Up @@ -49,7 +49,7 @@ stories.add(
step: 1
});
return (
<div style={{ width: '195px', border: '1px solid grey' }}>
<div style={{ width: '195px', border: '1px solid grey', paddingTop: '20px' }}>
<MockFilterSidePanelExample maxShowCount={maxShowCountValue} leeway={leewayValue} />
</div>
);
Expand Down

0 comments on commit abc8cc1

Please sign in to comment.