Skip to content

Commit

Permalink
feat: Adds a separator to Select All option (#22776)
Browse files Browse the repository at this point in the history
  • Loading branch information
michael-s-molina authored Jan 19, 2023
1 parent af73c59 commit 1fe4a71
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 2 deletions.
2 changes: 1 addition & 1 deletion superset-frontend/.storybook/preview.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -87,5 +87,5 @@ addParameters({
],
},
},
controls: { expanded: true },
controls: { expanded: true, sort: 'alpha' },
});
1 change: 1 addition & 0 deletions superset-frontend/src/components/Select/Select.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -467,6 +467,7 @@ const Select = forwardRef(
{selectAllEnabled && (
<Option
id="select-all"
className="select-all"
key={SELECT_ALL_VALUE}
value={SELECT_ALL_VALUE}
>
Expand Down
4 changes: 3 additions & 1 deletion superset-frontend/src/components/Select/styles.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -52,7 +52,9 @@ export const StyledSelect = styled(AntdSelect, {
.ant-select-arrow .anticon:not(.ant-select-suffix) {
pointer-events: none;
}
.select-all {
border-bottom: 1px solid ${theme.colors.grayscale.light3};
}
${
oneLine &&
`
Expand Down

0 comments on commit 1fe4a71

Please sign in to comment.