Lint

palantir · Feb 20, 2019 · 373765e · 373765e
1 parent 3dcbb01
commit 373765e
Show file tree

Hide file tree

Showing 18 changed files with 69 additions and 81 deletions.
diff --git a/packages/core/src/components/popover/popover.tsx b/packages/core/src/components/popover/popover.tsx
@@ -507,10 +507,7 @@ export class Popover extends AbstractPureComponent<IPopoverProps, IPopoverState>
     }
 
     private isArrowEnabled() {
-        const {
-            minimal,
-            modifiers: { arrow },
-        } = this.props;
+        const { minimal, modifiers: { arrow } } = this.props;
         // omitting `arrow` from `modifiers` uses Popper default, which does show an arrow.
         return !minimal && (arrow == null || arrow.enabled);
     }

diff --git a/packages/core/src/components/slider/handle.tsx b/packages/core/src/components/slider/handle.tsx
@@ -223,12 +223,8 @@ export class Handle extends AbstractPureComponent<IInternalHandleProps, IHandleS
         const handleRect = handleElement.getBoundingClientRect();
 
         const sizeKey = vertical
-            ? useOppositeDimension
-                ? "width"
-                : "height"
-            : useOppositeDimension
-            ? "height"
-            : "width";
+            ? useOppositeDimension ? "width" : "height"
+            : useOppositeDimension ? "height" : "width";
 
         // "bottom" value seems to be consistently incorrect, so explicitly
         // calculate it using the window offset instead.

diff --git a/packages/core/src/components/slider/multiSlider.tsx b/packages/core/src/components/slider/multiSlider.tsx
@@ -432,8 +432,9 @@ function getSortedInteractiveHandleProps(props: IChildrenProps): IHandleProps[]
 }
 
 function getSortedHandleProps({ children }: IChildrenProps, predicate: (props: IHandleProps) => boolean = () => true) {
-    const maybeHandles = React.Children.map(children, child =>
-        Utils.isElementOfType(child, MultiSlider.Handle) && predicate(child.props) ? child.props : null,
+    const maybeHandles = React.Children.map(
+        children,
+        child => (Utils.isElementOfType(child, MultiSlider.Handle) && predicate(child.props) ? child.props : null),
     );
     let handles = maybeHandles != null ? maybeHandles : [];
     handles = handles.filter(handle => handle !== null);

diff --git a/packages/core/src/components/spinner/spinner.tsx b/packages/core/src/components/spinner/spinner.tsx
@@ -81,7 +81,7 @@ export class Spinner extends AbstractPureComponent<ISpinnerProps, {}> {
         );
 
         // keep spinner track width consistent at all sizes (down to about 10px).
-        const strokeWidth = Math.min(MIN_STROKE_WIDTH, (STROKE_WIDTH * Spinner.SIZE_LARGE) / size);
+        const strokeWidth = Math.min(MIN_STROKE_WIDTH, STROKE_WIDTH * Spinner.SIZE_LARGE / size);
 
         const strokeOffset = PATH_LENGTH - PATH_LENGTH * (value == null ? 0.25 : clamp(value, 0, 1));
 

diff --git a/packages/core/test/popover/popoverTests.tsx b/packages/core/test/popover/popoverTests.tsx
@@ -597,14 +597,16 @@ describe("<Popover>", () => {
         afterEach(() => root.detach());
 
         it("shows tooltip on hover", () => {
-            root.find(`.${Classes.POPOVER_TARGET}`)
+            root
+                .find(`.${Classes.POPOVER_TARGET}`)
                 .last()
                 .simulate("mouseenter");
             assert.lengthOf(root.find(`.${Classes.TOOLTIP}`), 1);
         });
 
         it("shows popover on click", () => {
-            root.find(`.${Classes.POPOVER_TARGET}`)
+            root
+                .find(`.${Classes.POPOVER_TARGET}`)
                 .first()
                 .simulate("click");
             assert.lengthOf(root.find(`.${Classes.POPOVER}`), 1);

diff --git a/packages/datetime/src/datePicker.tsx b/packages/datetime/src/datePicker.tsx
@@ -188,9 +188,7 @@ export class DatePicker extends AbstractPureComponent<IDatePickerProps, IDatePic
     private disabledDays = (day: Date) => !DateUtils.isDayInRange(day, [this.props.minDate, this.props.maxDate]);
 
     private getDisabledDaysModifier = () => {
-        const {
-            dayPickerProps: { disabledDays },
-        } = this.props;
+        const { dayPickerProps: { disabledDays } } = this.props;
 
         return Array.isArray(disabledDays) ? [this.disabledDays, ...disabledDays] : [this.disabledDays, disabledDays];
     };

diff --git a/packages/datetime/src/dateRangePicker.tsx b/packages/datetime/src/dateRangePicker.tsx
@@ -144,10 +144,7 @@ export class DateRangePicker extends AbstractPureComponent<IDateRangePickerProps
         [`${SELECTED_RANGE_MODIFIER}-end`]: day => DateUtils.areSameDay(this.state.value[1], day),
 
         [HOVERED_RANGE_MODIFIER]: day => {
-            const {
-                hoverValue,
-                value: [selectedStart, selectedEnd],
-            } = this.state;
+            const { hoverValue, value: [selectedStart, selectedEnd] } = this.state;
             if (selectedStart == null && selectedEnd == null) {
                 return false;
             }
@@ -266,9 +263,7 @@ export class DateRangePicker extends AbstractPureComponent<IDateRangePickerProps
     private disabledDays = (day: Date) => !DateUtils.isDayInRange(day, [this.props.minDate, this.props.maxDate]);
 
     private getDisabledDaysModifier = () => {
-        const {
-            dayPickerProps: { disabledDays },
-        } = this.props;
+        const { dayPickerProps: { disabledDays } } = this.props;
 
         return disabledDays instanceof Array ? [this.disabledDays, ...disabledDays] : [this.disabledDays, disabledDays];
     };

diff --git a/packages/datetime/test/dateInputTests.tsx b/packages/datetime/test/dateInputTests.tsx
@@ -124,7 +124,8 @@ describe("<DateInput>", () => {
         const defaultValue = new Date(2018, Months.FEBRUARY, 6, 15, 0, 0, 0);
         const { root, changeSelect } = wrap(<DateInput {...DATE_FORMAT} defaultValue={defaultValue} />);
         root.setState({ isOpen: true });
-        root.find("input")
+        root
+            .find("input")
             .simulate("focus")
             .simulate("blur");
         changeSelect(Classes.DATEPICKER_MONTH_SELECT, Months.FEBRUARY);
@@ -135,7 +136,8 @@ describe("<DateInput>", () => {
         const defaultValue = new Date(2018, Months.FEBRUARY, 6, 15, 0, 0, 0);
         const { root, changeSelect } = wrap(<DateInput {...DATE_FORMAT} defaultValue={defaultValue} />);
         root.setState({ isOpen: true });
-        root.find("input")
+        root
+            .find("input")
             .simulate("focus")
             .simulate("blur");
         changeSelect(Classes.DATEPICKER_YEAR_SELECT, 2016);

diff --git a/packages/datetime/test/datePickerTests.tsx b/packages/datetime/test/datePickerTests.tsx
@@ -137,7 +137,8 @@ describe("<DatePicker>", () => {
             it("calls onMonthChange on button next click", () => {
                 const onMonthChange = sinon.spy();
                 const { root } = wrap(<DatePicker defaultValue={defaultValue} dayPickerProps={{ onMonthChange }} />);
-                root.find(".DayPicker-NavButton--next")
+                root
+                    .find(".DayPicker-NavButton--next")
                     .first()
                     .simulate("click");
                 assert.isTrue(onMonthChange.called);
@@ -146,7 +147,8 @@ describe("<DatePicker>", () => {
             it("calls onMonthChange on button prev click", () => {
                 const onMonthChange = sinon.spy();
                 const { root } = wrap(<DatePicker defaultValue={defaultValue} dayPickerProps={{ onMonthChange }} />);
-                root.find(".DayPicker-NavButton--prev")
+                root
+                    .find(".DayPicker-NavButton--prev")
                     .first()
                     .simulate("click");
                 assert.isTrue(onMonthChange.called);
@@ -155,7 +157,8 @@ describe("<DatePicker>", () => {
             it("calls onMonthChange on month select change", () => {
                 const onMonthChange = sinon.spy();
                 const { root } = wrap(<DatePicker defaultValue={defaultValue} dayPickerProps={{ onMonthChange }} />);
-                root.find({ className: Classes.DATEPICKER_MONTH_SELECT })
+                root
+                    .find({ className: Classes.DATEPICKER_MONTH_SELECT })
                     .first()
                     .find("select")
                     .simulate("change");
@@ -165,7 +168,8 @@ describe("<DatePicker>", () => {
             it("calls onMonthChange on year select change", () => {
                 const onMonthChange = sinon.spy();
                 const { root } = wrap(<DatePicker defaultValue={defaultValue} dayPickerProps={{ onMonthChange }} />);
-                root.find({ className: Classes.DATEPICKER_YEAR_SELECT })
+                root
+                    .find({ className: Classes.DATEPICKER_YEAR_SELECT })
                     .first()
                     .find("select")
                     .simulate("change");
@@ -499,7 +503,8 @@ describe("<DatePicker>", () => {
                 />,
             );
             assert.isTrue(onChangeSpy.notCalled);
-            root.find(`.${Classes.TIMEPICKER_ARROW_BUTTON}.${Classes.TIMEPICKER_HOUR}`)
+            root
+                .find(`.${Classes.TIMEPICKER_ARROW_BUTTON}.${Classes.TIMEPICKER_HOUR}`)
                 .first()
                 .simulate("click");
             assert.isTrue(onChangeSpy.calledOnce);
@@ -553,7 +558,8 @@ describe("<DatePicker>", () => {
 
     it("selects the current day when Today is clicked", () => {
         const { root } = wrap(<DatePicker showActionsBar={true} />);
-        root.find({ className: Classes.DATEPICKER_FOOTER })
+        root
+            .find({ className: Classes.DATEPICKER_FOOTER })
             .find(Button)
             .first()
             .simulate("click");
@@ -568,7 +574,8 @@ describe("<DatePicker>", () => {
     it("clears the value when Clear is clicked", () => {
         const { getDay, root } = wrap(<DatePicker showActionsBar={true} />);
         getDay().simulate("click");
-        root.find({ className: Classes.DATEPICKER_FOOTER })
+        root
+            .find({ className: Classes.DATEPICKER_FOOTER })
             .find(Button)
             .last()
             .simulate("click");

diff --git a/packages/datetime/test/dateRangePickerTests.tsx b/packages/datetime/test/dateRangePickerTests.tsx
@@ -714,7 +714,8 @@ describe("<DateRangePicker>", () => {
         describe("when only end date is defined", () => {
             it("should show a hovered range of [end, day] if day > end", () => {
                 const { left, assertHoveredDays } = render();
-                left.clickDay(14)
+                left
+                    .clickDay(14)
                     .clickDay(18)
                     .clickDay(14) // deselect start date
                     .mouseEnterDay(22);
@@ -723,7 +724,8 @@ describe("<DateRangePicker>", () => {
 
             it("should show a hovered range of [null, null] if day === end", () => {
                 const { left, assertHoveredDays } = render();
-                left.clickDay(14)
+                left
+                    .clickDay(14)
                     .clickDay(18)
                     .clickDay(14)
                     .mouseEnterDay(18);
@@ -732,7 +734,8 @@ describe("<DateRangePicker>", () => {
 
             it("should show a hovered range of [day, end] if day < end", () => {
                 const { left, assertHoveredDays } = render();
-                left.clickDay(14)
+                left
+                    .clickDay(14)
                     .clickDay(18)
                     .clickDay(14)
                     .mouseEnterDay(14);
@@ -743,47 +746,53 @@ describe("<DateRangePicker>", () => {
         describe("when both start and end date are defined", () => {
             it("should show a hovered range of [null, end] if day === start", () => {
                 const { left, assertHoveredDays } = render();
-                left.clickDay(14)
+                left
+                    .clickDay(14)
                     .clickDay(18)
                     .mouseEnterDay(14);
                 assertHoveredDays(null, 18);
             });
 
             it("should show a hovered range of [start, null] if day === end", () => {
                 const { left, assertHoveredDays } = render();
-                left.clickDay(14)
+                left
+                    .clickDay(14)
                     .clickDay(18)
                     .mouseEnterDay(18);
                 assertHoveredDays(14, null);
             });
 
             it("should show a hovered range of [day, null] if start < day < end", () => {
                 const { left, assertHoveredDays } = render();
-                left.clickDay(14)
+                left
+                    .clickDay(14)
                     .clickDay(18)
                     .mouseEnterDay(16);
                 assertHoveredDays(16, null);
             });
 
             it("should show a hovered range of [day, null] if day < start", () => {
                 const { left, assertHoveredDays } = render();
-                left.clickDay(14)
+                left
+                    .clickDay(14)
                     .clickDay(18)
                     .mouseEnterDay(10);
                 assertHoveredDays(10, null);
             });
 
             it("should show a hovered range of [day, null] if day > end", () => {
                 const { left, assertHoveredDays } = render();
-                left.clickDay(14)
+                left
+                    .clickDay(14)
                     .clickDay(18)
                     .mouseEnterDay(22);
                 assertHoveredDays(22, null);
             });
 
             it("should show a hovered range of [null, null] if start === day === end", () => {
                 const { left, assertHoveredDays } = render({ allowSingleDayRange: true });
-                left.clickDay(14)
+                left
+                    .clickDay(14)
                     .clickDay(14)
                     .mouseEnterDay(14);
                 assertHoveredDays(null, null);
@@ -975,7 +984,8 @@ describe("<DateRangePicker>", () => {
         it("onHoverChange fired with `undefined` on mouseleave within a day", () => {
             const { left } = render({ initialMonth: new Date(2015, Months.JANUARY, 1) });
             assert.isTrue(onHoverChangeSpy.notCalled);
-            left.clickDay(1)
+            left
+                .clickDay(1)
                 .findDay(5)
                 .simulate("mouseleave");
             assert.isTrue(onHoverChangeSpy.calledTwice);
@@ -1018,7 +1028,8 @@ describe("<DateRangePicker>", () => {
 
         it("deselects endpoint when an endpoint of the current selection is clicked", () => {
             const { assertSelectedDays, left } = render({ initialMonth: new Date(2015, Months.JANUARY, 1) });
-            left.clickDay(10)
+            left
+                .clickDay(10)
                 .clickDay(14)
                 .clickDay(10);
             assertSelectedDays(14);

diff --git a/packages/datetime/test/dateTimePickerTests.tsx b/packages/datetime/test/dateTimePickerTests.tsx
@@ -55,7 +55,8 @@ describe("<DateTimePicker>", () => {
             />,
         );
         assert.isTrue(onChangeSpy.notCalled);
-        root.find(`.${Classes.TIMEPICKER_ARROW_BUTTON}.${Classes.TIMEPICKER_HOUR}`)
+        root
+            .find(`.${Classes.TIMEPICKER_ARROW_BUTTON}.${Classes.TIMEPICKER_HOUR}`)
             .first()
             .simulate("click");
         assert.isTrue(onChangeSpy.calledOnce);
@@ -79,7 +80,8 @@ describe("<DateTimePicker>", () => {
             <DateTimePicker defaultValue={defaultValue} timePickerProps={{ showArrowButtons: true }} />,
         );
         getDay(5).simulate("click");
-        root.find(`.${Classes.TIMEPICKER_ARROW_BUTTON}.${Classes.TIMEPICKER_HOUR}`)
+        root
+            .find(`.${Classes.TIMEPICKER_ARROW_BUTTON}.${Classes.TIMEPICKER_HOUR}`)
             .first()
             .simulate("click");
         getDay(15).simulate("click");

diff --git a/packages/docs-app/src/components/colorPalettes.tsx b/packages/docs-app/src/components/colorPalettes.tsx
@@ -58,9 +58,7 @@ const ColorSwatch: React.SFC<{ colorName: string; hexCode: string }> = ({ colorN
 // vertical list of swatches for each color
 const ColorPalette: React.SFC<{ colors: string[] }> = ({ colors }) => (
     <div className={classNames("docs-color-palette", { "docs-color-palette-single": colors.length === 1 })}>
-        {colors.map((name, i) => (
-            <ColorSwatch colorName={name} hexCode={getHexCode(name)} key={i} />
-        ))}
+        {colors.map((name, i) => <ColorSwatch colorName={name} hexCode={getHexCode(name)} key={i} />)}
     </div>
 );
 
@@ -94,9 +92,7 @@ export const ColorBar: React.SFC<{ colors: string[] }> = ({ colors }) => {
 function createPaletteBook(palettes: string[][], className?: string): React.SFC<{}> {
     return () => (
         <section className={classNames("docs-color-book", className)}>
-            {palettes.map((palette, index) => (
-                <ColorPalette colors={palette} key={index} />
-            ))}
+            {palettes.map((palette, index) => <ColorPalette colors={palette} key={index} />)}
         </section>
     );
 }

diff --git a/packages/docs-app/src/examples/core-examples/panelStackExample.tsx b/packages/docs-app/src/examples/core-examples/panelStackExample.tsx
@@ -30,11 +30,7 @@ export class PanelStackExample extends React.PureComponent<IExampleProps, IPanel
         const stackList = (
             <>
                 <H5>Current stack</H5>
-                <UL>
-                    {this.state.currentPanelStack.map((p, i) => (
-                        <li key={i}>{p.title}</li>
-                    ))}
-                </UL>
+                <UL>{this.state.currentPanelStack.map((p, i) => <li key={i}>{p.title}</li>)}</UL>
             </>
         );
         return (

diff --git a/packages/docs-app/src/examples/datetime-examples/common/formatSelect.tsx b/packages/docs-app/src/examples/datetime-examples/common/formatSelect.tsx
@@ -25,9 +25,7 @@ export class FormatSelect extends React.PureComponent<IFormatSelectProps> {
         const value = FORMATS.indexOf(this.props.format);
         return (
             <RadioGroup label="Date format" onChange={this.handleChange} selectedValue={value}>
-                {FORMATS.map((format, index) => (
-                    <Radio key={index} label={format.placeholder} value={index} />
-                ))}
+                {FORMATS.map((format, index) => <Radio key={index} label={format.placeholder} value={index} />)}
             </RadioGroup>
         );
     }

diff --git a/packages/docs-app/src/examples/datetime-examples/dateRangePickerExample.tsx b/packages/docs-app/src/examples/datetime-examples/dateRangePickerExample.tsx
@@ -173,9 +173,7 @@ export class DateRangePickerExample extends React.PureComponent<IExampleProps, I
             <Label>
                 {label}
                 <HTMLSelect value={selectedValue} onChange={onChange}>
-                    {options.map((opt, i) => (
-                        <option key={i} value={i} label={opt.label} />
-                    ))}
+                    {options.map((opt, i) => <option key={i} value={i} label={opt.label} />)}
                 </HTMLSelect>
             </Label>
         );