Button icon variant

Summary: Allow setting the icon variant for icons on buttons

Reviewed By: jknoxville

Differential Revision: D10101603

fbshipit-source-id: 344afda2296479ffb8449a7ad8172b35f0dc95e4

src/ui/components/Button.js

@@ -203,6 +203,7 @@ type Props = {
    * Whether the button should render depressed into its socket
    */
   depressed?: boolean,
+  iconVariant?: 'filled' | 'outline',
 };
 
 type State = {
@@ -300,6 +301,7 @@ class Button extends React.Component<
       selected,
       iconSize,
       windowIsFocused,
+      iconVariant,
       ...props
     } = this.props;
     const {active} = this.state;
@@ -324,10 +326,9 @@ class Button extends React.Component<
       iconComponent = (
         <Icon
           name={icon}
-          size={
+          size={iconSize || this.props.compact === true ? 12 : 16}
-            iconSize != null ? iconSize : this.props.compact === true ? 12 : 16
-          }
           color={color}
+          variant={iconVariant || 'filled'}
           hasText={Boolean(children)}
         />
       );