Skip to content

Commit

Permalink
Remove TwoTone icons (apache#19666)
Browse files Browse the repository at this point in the history
  • Loading branch information
geido authored and philipher29 committed Jun 9, 2022
1 parent 531b857 commit 5ac2e38
Show file tree
Hide file tree
Showing 3 changed files with 1 addition and 6 deletions.
1 change: 1 addition & 0 deletions superset-frontend/src/components/Icons/AntdEnhanced.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,7 @@ import { StyledIcon } from './Icon';
import IconType from './IconType';

const AntdEnhancedIcons = Object.keys(AntdIcons)
.filter(k => !k.includes('TwoTone'))
.map(k => ({
[k]: (props: IconType) => (
<StyledIcon component={AntdIcons[k]} {...props} />
Expand Down
1 change: 0 additions & 1 deletion superset-frontend/src/components/Icons/IconType.ts
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,6 @@ import { IconComponentProps } from '@ant-design/icons/lib/components/Icon';
type AntdIconType = IconComponentProps;
type IconType = AntdIconType & {
iconColor?: string;
twoToneColor?: string;
iconSize?: 'xs' | 's' | 'm' | 'l' | 'xl' | 'xxl';
};

Expand Down
5 changes: 0 additions & 5 deletions superset-frontend/src/components/Icons/Icons.stories.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -78,11 +78,6 @@ InteractiveIcons.argTypes = {
defaultValue: null,
control: { type: 'select', options: palette },
},
// @TODO twoToneColor is being ignored
twoToneColor: {
defaultValue: null,
control: { type: 'select', options: palette },
},
theme: {
table: {
disable: true,
Expand Down

0 comments on commit 5ac2e38

Please sign in to comment.