@sproutsocial/racine 11.4.2-input-beta.0 → 11.5.0

package/CHANGELOG.md

@@ -1,5 +1,11 @@
 # Change Log
 
+## 11.5.0
+
+### Minor Changes
+
+- fa8a45d: Dependency upgrades: upgraded severe vulnerabilities as well as storybook and jest suite fixes. Upgraded styled components from beta to stable v5.2.3.
+
 ## 11.4.1
 
 ### Patch Changes

package/__flow__/Button/index.js

@@ -84,4 +84,6 @@ const Button = ({
   );
 };
 
+Button.displayName = "Button";
+
 export default Button;

package/__flow__/Button/index.stories.js

@@ -1,5 +1,4 @@
 import React from "react";
-import { boolean, text, number } from "@storybook/addon-knobs";
 import Button from "./index";
 import Icon from "../Icon";
 import Box from "../Box";
@@ -8,155 +7,140 @@ export default {
   title: "Button",
 };
 
-export const defaultStory = () => (
-  <Button
-    appearance={text("appearance", "default")}
-    onClick={() => alert("Testing...")}
-  >
+export const defaultStory = (args) => (
+  <Button {...args} onClick={() => alert("Testing...")}>
     Default
   </Button>
 );
 
+defaultStory.args = { appearance: "default" };
+
 defaultStory.story = {
   name: "Default",
 };
 
-export const primary = () => (
-  <Button
-    appearance={text("appearance", "primary")}
-    onClick={() => alert("Testing...")}
-  >
+export const primary = (args) => (
+  <Button {...args} onClick={() => alert("Testing...")}>
     Primary Button
   </Button>
 );
 
+primary.args = { appearance: "primary" };
+
 primary.story = {
   name: "Primary",
 };
 
-export const secondary = () => (
-  <Button appearance={text("appearance", "secondary")}>Secondary Button</Button>
-);
+export const secondary = (args) => <Button {...args}>Secondary Button</Button>;
+
+secondary.args = { appearance: "secondary" };
 
 secondary.story = {
   name: "Secondary",
 };
 
-export const destructive = () => (
-  <Button appearance={text("appearance", "destructive")}>
-    Destructive Button
-  </Button>
+export const destructive = (args) => (
+  <Button {...args}>Destructive Button</Button>
 );
 
+destructive.args = { appearance: "destructive" };
 destructive.story = {
   name: "Destructive",
 };
 
-export const placeholder = () => (
-  <Button appearance={text("appearance", "placeholder")}>
-    Placeholder Button
-  </Button>
+export const placeholder = (args) => (
+  <Button {...args}>Placeholder Button</Button>
 );
 
+placeholder.args = { appearance: "placeholder" };
 placeholder.story = {
   name: "Placeholder",
 };
 
-export const largeButton = () => (
-  <Button
-    appearance={text("appearance", "primary")}
-    size={text("size", "large")}
-  >
-    Large Button
-  </Button>
-);
+export const largeButton = (args) => <Button {...args}>Large Button</Button>;
+largeButton.args = { size: "large", appearance: "primary" };
 
 largeButton.story = {
   name: "Large button",
 };
 
-export const pillButton = () => (
+export const pillButton = (args) => (
   <Box bg="container.background.base" p={400}>
-    <Button appearance={text("shape", "pill")}>
+    <Button {...args}>
       <Icon name="reply" mr={350} />
       Pill Button
     </Button>
   </Box>
 );
-
+pillButton.args = { appearance: "pill" };
 pillButton.story = {
   name: "Pill button",
 };
 
-export const pillIconOnlyButton = () => (
+export const pillIconOnlyButton = (args) => (
   <Box bg="container.background.base" p={400}>
-    <Button appearance={text("shape", "pill")} ariaLabel="This is a label">
+    <Button {...args} ariaLabel="This is a label">
       <Icon name="circle-check-outline" />
     </Button>
   </Box>
 );
 
+pillIconOnlyButton.args = { appearance: "pill" };
 pillIconOnlyButton.story = {
   name: "Pill icon only button",
 };
 
-export const activeButton = () => (
-  <Button
-    appearance={text("appearance", "secondary")}
-    active={boolean("active", true)}
-  >
-    Active Button
-  </Button>
-);
+export const activeButton = (args) => <Button {...args}>Active Button</Button>;
 
+activeButton.args = { appearance: "secondary", active: true };
 activeButton.story = {
   name: "Active button",
 };
 
-export const buttonAsALink = () => (
-  <Button
-    href={text("href", "http://sproutsocial.style")}
-    external={boolean("external", true)}
-    appearance={text("appearance", "primary")}
-  >
-    Button using anchor element
-  </Button>
+export const buttonAsALink = (args) => (
+  <Button {...args}>Button using anchor element</Button>
 );
-
+buttonAsALink.args = {
+  appearance: "primary",
+  external: true,
+  href: "http://sproutsocial.style",
+};
 buttonAsALink.story = {
   name: "Button as a link",
 };
 
-export const disabledButton = () => (
-  <Button
-    appearance={text("appearance", "primary")}
-    disabled={text("disabled", "true")}
-  >
-    This Button is disabled
-  </Button>
+export const disabledButton = (args) => (
+  <Button {...args}>This Button is disabled</Button>
 );
-
+disabledButton.args = {
+  appearance: "primary",
+  disabled: true,
+};
 disabledButton.story = {
   name: "Disabled button",
 };
 
-export const fullWidthButton = () => (
-  <Button appearance={text("appearance", "primary")} width={number("width", 1)}>
-    Full-Width Button
-  </Button>
+export const fullWidthButton = (args) => (
+  <Button {...args}>Full-Width Button</Button>
 );
-
+fullWidthButton.args = {
+  appearance: "primary",
+  width: 1,
+};
 fullWidthButton.story = {
   name: "Full width button",
 };
 
-export const withIcon = () => (
-  <Button appearance={text("appearance", "secondary")}>
+export const withIcon = (args) => (
+  <Button {...args}>
     <Icon name="twitter" mr={350} />
     Secondary Button
   </Button>
 );
 
+withIcon.args = {
+  appearance: "secondary",
+};
 withIcon.story = {
   name: "With icon",
 };

package/__flow__/Button/styles.js

@@ -99,7 +99,7 @@ const Container: StyledComponent<any, TypeTheme, *> = styled.button`
   ${LAYOUT}
   ${COMMON}
 `;
-
+Container.displayName = "Button-Container";
 export default Container;
 
 //${props.theme.mode === "dark" ? "screen" : "multiply"}

package/__flow__/Checkbox/styles.js

@@ -182,90 +182,90 @@ const getIcon = (type, color) => {
 
 // eslint-disable-next-line prettier/prettier
 export const CheckboxContainer: StyledComponent<any, TypeTheme, *> = styled.span(
-  (props) => css`
-    display: inline-flex;
-    align-items: center;
-    box-sizing: border-box;
-    position: relative;
-    transition: all ${props.theme.duration.fast} ${props.theme.easing.ease_in};
-
-    @supports (-webkit-appearance: none) {
-      &:before {
-        /* stylelint-disable */
-        content: url("data:image/svg+xml;utf8,${getIcon(
-          props.indeterminate ? "indeterminate" : "check",
-
-          props.checked
-            ? props.theme.colors.form.background.base
-            : props.theme.colors.form.border.base
-        )}");
-        opacity: ${props.checked ? 1 : 0};
-        position: absolute;
-        width: ${props.theme.space[400]};
-        height: ${props.theme.space[400]};
-        text-align: center;
-        transform: translateY(1px);
-        line-height: 1;
-        margin: auto;
-        pointer-events: none;
-        transition: ${props.theme.duration.fast}
-          ${props.theme.easing.ease_inout};
-      }
-
-      &:hover:before {
-        opacity: ${props.disabled && !props.checked ? 0 : 1};
-      }
-
-      ${props.disabled &&
-      css`
-        opacity: 0.4;
-      `}
-
-      input[type='checkbox'] {
-        box-sizing: border-box;
-        appearance: none;
-        margin: 0;
-        padding: 0;
-        width: ${props.theme.space[400]};
-        height: ${props.theme.space[400]};
-        border: 1px solid ${props.theme.colors.form.border.base};
-        border-radius: 4px;
-        background-color: ${props.theme.colors.form.background.base};
-        transition: all ${props.theme.duration.fast}
-          ${props.theme.easing.ease_in};
-        cursor: ${props.disabled ? "not-allowed" : "pointer"};
-        flex-shrink: 0;
-
-        &:not(:checked) {
-          ${!props.indeterminate &&
-          css`
-            border-color: ${props.theme.colors
-              .neutral[300]} !important; /* We don't want the focus ring to remove the border */
-            background-color: ${props.theme.colors.form.background.base};
-          `}
+    (props) => css`
+      display: inline-flex;
+      align-items: center;
+      box-sizing: border-box;
+      position: relative;
+      transition: all ${props.theme.duration.fast} ${props.theme.easing.ease_in};
+
+      @supports (-webkit-appearance: none) {
+        &:before {
+          /* stylelint-disable */
+          content: url("data:image/svg+xml;utf8,${getIcon(
+            props.indeterminate ? "indeterminate" : "check",
+
+            props.checked
+              ? props.theme.colors.form.background.base
+              : props.theme.colors.form.border.base
+          )}");
+          opacity: ${props.checked ? 1 : 0};
+          position: absolute;
+          width: ${props.theme.space[400]};
+          height: ${props.theme.space[400]};
+          text-align: center;
+          transform: translateY(1px);
+          line-height: 1;
+          margin: auto;
+          pointer-events: none;
+          transition: ${props.theme.duration.fast}
+            ${props.theme.easing.ease_inout};
         }
 
-        &:checked {
-          border-color: ${props.theme.colors.form.border.selected};
-          background-color: ${props.theme.colors.form.background.selected};
+        &:hover:before {
+          opacity: ${props.disabled && !props.checked ? 0 : 1};
         }
 
-        ${props.indeterminate &&
-        props.checked &&
+        ${props.disabled &&
         css`
-          border-color: ${props.theme.colors.form.border.selected} !important;
-          background-color: ${props.theme.colors.form.background
-            .selected} !important;
+          opacity: 0.4;
         `}
 
-        &:focus {
-          ${focusRing}
+        input[type='checkbox'] {
+          box-sizing: border-box;
+          appearance: none;
+          margin: 0;
+          padding: 0;
+          width: ${props.theme.space[400]};
+          height: ${props.theme.space[400]};
+          border: 1px solid ${props.theme.colors.form.border.base};
+          border-radius: 4px;
+          background-color: ${props.theme.colors.form.background.base};
+          transition: all ${props.theme.duration.fast}
+            ${props.theme.easing.ease_in};
+          cursor: ${props.disabled ? "not-allowed" : "pointer"};
+          flex-shrink: 0;
+
+          &:not(:checked) {
+            ${!props.indeterminate &&
+            css`
+              border-color: ${props.theme.colors
+                .neutral[300]} !important; /* We don't want the focus ring to remove the border */
+              background-color: ${props.theme.colors.form.background.base};
+            `}
+          }
+
+          &:checked {
+            border-color: ${props.theme.colors.form.border.selected};
+            background-color: ${props.theme.colors.form.background.selected};
+          }
+
+          ${props.indeterminate &&
+          props.checked &&
+          css`
+            border-color: ${props.theme.colors.form.border.selected} !important;
+            background-color: ${props.theme.colors.form.background
+              .selected} !important;
+          `}
+
+          &:focus {
+            ${focusRing}
+          }
         }
       }
-    }
 
-    ${COMMON}
-  `
-);
+      ${COMMON}
+    `
+  );
 
 export default Container;

package/__flow__/Collapsible/index.js

@@ -73,9 +73,8 @@ const Trigger = ({ children, ...rest }) => {
 };
 
 const Panel = ({ children, ...rest }) => {
-  const { isOpen, id, offset, collapsedHeight, openHeight } = useContext(
-    CollapsibleContext
-  );
+  const { isOpen, id, offset, collapsedHeight, openHeight } =
+    useContext(CollapsibleContext);
   const ref = useRef();
   const measurement = useMeasure(ref);
   const [isHidden, setIsHidden] = useState(undefined);

package/__flow__/EmptyState/index.test.js

@@ -105,7 +105,7 @@ describe("EmptyState", () => {
       <EmptyState
         media={
           <Image
-            alt="No assets matching your
 search or filters"
+            alt="No assets matching your
  search or filters"
             src="https://cl.ly/db498c7682df/download/analytics.svg"
             m={0}
           />

package/__flow__/Image/index.js

@@ -80,16 +80,8 @@ export default class Image extends React.Component<TypeProps, TypeState> {
   };
 
   render() {
-    const {
-      alt,
-      title,
-      onClick,
-      onError,
-      onLoad,
-      src,
-      qa,
-      ...rest
-    } = this.props;
+    const { alt, title, onClick, onError, onLoad, src, qa, ...rest } =
+      this.props;
 
     return (
       <ImageContainer

package/__flow__/Input/index.js

@@ -74,10 +74,6 @@ type TypeProps = {
   appearance?: "primary" | "secondary",
 };
 
-type TypeState = {
-  hasValue: boolean,
-};
-
 // Using Context so that Input's Input.ClearButton-specific props can be passed to Input.ClearButton,
 // regardless of whether it is manually included as elemAfter or automatically included for type="search" Inputs.
 type TypeInputContext = $Shape<{
@@ -158,16 +154,7 @@ const isClearButton = (elem: any) => {
   return false;
 };
 
-class Input extends React.Component<TypeProps, TypeState> {
-  constructor(props: TypeProps) {
-    super(props);
-    this.state = {
-      // tracking hasValue in state allows us to hide ClearButton when there is no value to clear
-      // for both controlled and uncontrolled inputs.
-      hasValue: !!props.value,
-    };
-  }
-
+class Input extends React.Component<TypeProps> {
   static defaultProps = {
     autoFocus: false,
     disabled: false,
@@ -190,10 +177,8 @@ class Input extends React.Component<TypeProps, TypeState> {
     this.props.onClear?.(e);
   };
 
-  handleChange = (e: SyntheticInputEvent<HTMLInputElement>) => {
+  handleChange = (e: SyntheticInputEvent<HTMLInputElement>) =>
     this.props.onChange?.(e, e.currentTarget.value);
-    this.updateState(e.currentTarget.value);
-  };
 
   handleFocus = (e: SyntheticFocusEvent<HTMLInputElement>) =>
     this.props.onFocus?.(e);
@@ -207,15 +192,6 @@ class Input extends React.Component<TypeProps, TypeState> {
   handlePaste = (e: SyntheticInputEvent<HTMLInputElement>) =>
     this.props.onPaste?.(e, e.currentTarget.value);
 
-  updateState = (value: string) => {
-    const hasValue = value !== "";
-    const hadValue = this.state.hasValue;
-    // Only update state if the value has changed to avoid unnecessary renders.
-    if (hasValue !== hadValue) {
-      this.setState({ hasValue });
-    }
-  };
-
   render() {
     const {
       autoComplete,
@@ -283,7 +259,7 @@ class Input extends React.Component<TypeProps, TypeState> {
         <InputContext.Provider
           value={{
             handleClear: this.handleClear,
-            hasValue: this.state.hasValue,
+            hasValue: !!value,
             clearButtonLabel,
             onClear,
             size,

package/__flow__/Input/index.stories.js

@@ -188,20 +188,6 @@ largeSearchInput.story = {
   name: "Large Search Input",
 };
 
-export const uncontrolledSearchInput = () => (
-  <Input
-    type="search"
-    placeholder={text("placeholder", "Please enter a value...")}
-    onClear={() => window.alert("Cleared!")}
-    clearButtonLabel={text("clearButtonLabel", "Clear search")}
-    ariaLabel={text("ariaLabel", "Descriptive label goes here")}
-  />
-);
-
-uncontrolledSearchInput.story = {
-  name: "Uncontrolled Search Input",
-};
-
 export const nonSearchClearButtonInput = () => {
   return (
     <Input

package/__flow__/Input/index.test.js

@@ -85,24 +85,6 @@ describe("Input", () => {
         expect(getByRole("button")).toBeTruthy();
       });
 
-      it("should render a clear button for an uncontrolled search Input only once it is typed in", () => {
-        const { getByRole, queryByRole } = render(
-          <Input
-            id="name"
-            name="name"
-            type="search"
-            onClear={jest.fn()}
-            clearButtonLabel="Clear search"
-          />
-        );
-        expect(queryByRole("button")).toBeFalsy();
-        const input = getByRole("searchbox");
-        userEvent.tab();
-        expect(input).toHaveFocus();
-        userEvent.keyboard("some text");
-        expect(getByRole("button")).toBeTruthy();
-      });
-
       it("should not render a clear button for search Inputs if there is no text", () => {
         const { queryByRole } = render(
           <Input

package/__flow__/Menu/__snapshots__/index.test.js.snap

@@ -14,12 +14,12 @@ exports[`Menu AsMenuButton should match snapshot 1`] = `
   fill: currentColor;
 }
 
-_:-ms-fullscreen .c3,
+_:-ms-fullscreen .c2,
 html .c3 {
   width: 16px;
 }
 
-.pdf-page .c3 {
+.pdf-page .c2 {
   width: 16px;
 }
 

package/__flow__/SegmentedControl/index.js

@@ -17,9 +17,8 @@ type TypeSegmentedControlContext = {
   onChange: (e: SyntheticInputEvent<HTMLInputElement>) => void,
 };
 
-const SegmentedControlContext = React.createContext<?TypeSegmentedControlContext>(
-  null
-);
+const SegmentedControlContext =
+  React.createContext<?TypeSegmentedControlContext>(null);
 
 type TypeSegmentedControlItemProps = {
   /** The value of this item. Should be unique among sibling items. */

package/__flow__/TableCell/index.js

@@ -22,15 +22,8 @@ export type TypeTableCell = {
  */
 export default class TableCell extends React.Component<TypeTableCell> {
   render() {
-    const {
-      id,
-      content,
-      colSpan,
-      width,
-      align,
-      children,
-      ...rest
-    } = this.props;
+    const { id, content, colSpan, width, align, children, ...rest } =
+      this.props;
 
     return (
       <Container

package/__flow__/ToggleHint/index.js

@@ -32,15 +32,8 @@ export default class ToggleHint extends React.Component<TypeProps> {
   };
 
   render() {
-    const {
-      icon,
-      isOpen,
-      openString,
-      closeString,
-      qa,
-      className,
-      ...rest
-    } = this.props;
+    const { icon, isOpen, openString, closeString, qa, className, ...rest } =
+      this.props;
 
     return (
       <Container

package/__flow__/setupTests.js

@@ -1,5 +1,5 @@
 import "mutationobserver-shim";
-import "jest-dom/extend-expect";
+import "@testing-library/jest-dom/extend-expect";
 import "jest-axe/extend-expect";
 import "babel-polyfill";
 import "jest-styled-components";

package/__flow__/systemProps/color.js

@@ -14,7 +14,8 @@ import type {
 
 // https://styled-system.com/table#color
 
-type TypeBackgroundColorSystemProp = TypeResponsiveBaseSystemProp<BackgroundColorProperty>;
+type TypeBackgroundColorSystemProp =
+  TypeResponsiveBaseSystemProp<BackgroundColorProperty>;
 export type TypeColorSystemProps = $ReadOnly<{|
   backgroundColor?: TypeBackgroundColorSystemProp,
   bg?: TypeBackgroundColorSystemProp,

package/__flow__/systemProps/tests/__snapshots__/layout.test.js.snap

@@ -1,19 +1,5 @@
 // Jest Snapshot v1, https://goo.gl/fbAQLP
 
-exports[`layout system props display 1`] = `
-.c0 {
-  display: -webkit-string;
-  display: string;
-}
-
-<div>
-  <div
-    class="c0"
-    display="string"
-  />
-</div>
-`;
-
 exports[`layout system props height 1`] = `
 .c0 {
   height: string;