From e886d772c731cabb5992650fca966e9786da5a03 Mon Sep 17 00:00:00 2001 From: Eric Liu Date: Sat, 18 Jul 2020 20:00:20 -0700 Subject: [PATCH] refactor: use $$restProps API - add ref prop for applicable components (#196) - add slot to Content Switcher `Switch` component (#183) - remove fillArray, css utilities --- .storybook/preview-body.html | 8 + .storybook/preview-head.html | 5 +- package.json | 2 +- .../Accordion/Accordion.Skeleton.svelte | 37 ++- .../Accordion/Accordion.Story.svelte | 95 +++---- src/components/Accordion/Accordion.stories.js | 31 ++- src/components/Accordion/Accordion.svelte | 23 +- src/components/Accordion/AccordionItem.svelte | 28 +- src/components/Accordion/index.js | 8 +- .../Breadcrumb/Breadcrumb.Skeleton.svelte | 19 +- .../Breadcrumb/Breadcrumb.Story.svelte | 63 ++--- .../Breadcrumb/Breadcrumb.stories.js | 16 +- src/components/Breadcrumb/Breadcrumb.svelte | 16 +- src/components/Breadcrumb/Breadcrumb.test.js | 40 +-- .../Breadcrumb/BreadcrumbItem.svelte | 20 +- src/components/Breadcrumb/index.js | 8 +- src/components/Button/Button.Skeleton.svelte | 23 +- src/components/Button/Button.Story.svelte | 72 +++-- src/components/Button/Button.stories.js | 72 ++--- src/components/Button/Button.svelte | 81 +++--- src/components/Button/index.js | 6 +- .../Checkbox/Checkbox.Skeleton.svelte | 18 +- src/components/Checkbox/Checkbox.Story.svelte | 70 ++--- src/components/Checkbox/Checkbox.stories.js | 30 +- src/components/Checkbox/Checkbox.svelte | 55 ++-- src/components/Checkbox/index.js | 6 +- .../CodeSnippet/CodeSnippet.Skeleton.svelte | 18 +- .../CodeSnippet/CodeSnippet.Story.svelte | 52 ++-- .../CodeSnippet/CodeSnippet.stories.js | 53 ++-- src/components/CodeSnippet/CodeSnippet.svelte | 102 ++++--- src/components/CodeSnippet/index.js | 6 +- src/components/ComboBox/ComboBox.Story.svelte | 82 +++--- src/components/ComboBox/ComboBox.stories.js | 34 +-- src/components/ComboBox/ComboBox.svelte | 66 +++-- src/components/ComboBox/index.js | 4 +- .../ComposedModal/ComposedModal.Story.svelte | 128 +++++---- .../ComposedModal/ComposedModal.stories.js | 176 +++++++----- .../ComposedModal/ComposedModal.svelte | 65 +++-- src/components/ComposedModal/ModalBody.svelte | 12 +- .../ComposedModal/ModalFooter.svelte | 18 +- .../ComposedModal/ModalHeader.svelte | 45 +-- src/components/ComposedModal/index.js | 10 +- .../ContentSwitcher.Story.svelte | 24 +- .../ContentSwitcher.stories.js | 16 +- .../ContentSwitcher/ContentSwitcher.svelte | 46 ++-- src/components/ContentSwitcher/Switch.svelte | 48 ++-- src/components/ContentSwitcher/index.js | 6 +- src/components/Copy/Copy.svelte | 53 ++-- src/components/Copy/index.js | 4 +- .../CopyButton/CopyButton.Story.svelte | 13 +- .../CopyButton/CopyButton.stories.js | 20 +- src/components/CopyButton/CopyButton.svelte | 54 +--- src/components/CopyButton/index.js | 4 +- .../DataTable/DataTable.Story.svelte | 112 ++++---- src/components/DataTable/DataTable.stories.js | 36 +-- src/components/DataTable/DataTable.svelte | 67 ++--- src/components/DataTable/Table.svelte | 31 ++- src/components/DataTable/TableBody.svelte | 8 +- src/components/DataTable/TableCell.svelte | 8 +- .../DataTable/TableContainer.svelte | 20 +- src/components/DataTable/TableHead.svelte | 8 +- src/components/DataTable/TableHeader.svelte | 48 ++-- src/components/DataTable/TableRow.svelte | 11 +- src/components/DataTable/index.js | 18 +- .../DataTableSkeleton/DataTable.stories.js | 18 +- .../DataTableSkeleton.Story.svelte | 4 +- .../DataTableSkeleton.svelte | 25 +- src/components/DataTableSkeleton/index.js | 4 +- .../DatePicker/DatePicker.Skeleton.svelte | 29 +- .../DatePicker/DatePicker.Story.svelte | 12 +- .../DatePicker/DatePicker.stories.js | 161 +++++++---- src/components/DatePicker/DatePicker.svelte | 69 +++-- .../DatePicker/DatePickerInput.svelte | 50 ++-- src/components/DatePicker/createCalendar.js | 66 +++-- src/components/DatePicker/index.js | 6 +- .../Dropdown/Dropdown.Skeleton.svelte | 19 +- src/components/Dropdown/Dropdown.Story.svelte | 21 +- src/components/Dropdown/Dropdown.stories.js | 50 ++-- src/components/Dropdown/Dropdown.svelte | 87 +++--- src/components/Dropdown/index.js | 6 +- .../FileUploader/FileUploader.Skeleton.svelte | 19 +- .../FileUploader/FileUploader.Story.svelte | 22 +- .../FileUploader/FileUploader.stories.js | 136 +++++---- .../FileUploader/FileUploader.svelte | 71 +++-- .../FileUploader/FileUploaderButton.svelte | 51 ++-- .../FileUploaderDropContainer.svelte | 56 ++-- .../FileUploader/FileUploaderItem.svelte | 37 ++- src/components/FileUploader/Filename.svelte | 37 ++- src/components/FileUploader/index.js | 12 +- src/components/Form/Form.Story.svelte | 82 ++++-- src/components/Form/Form.stories.js | 16 +- src/components/Form/Form.svelte | 14 +- src/components/Form/index.js | 4 +- src/components/FormGroup/FormGroup.svelte | 19 +- src/components/FormGroup/index.js | 4 +- src/components/FormItem/FormItem.Story.svelte | 6 +- src/components/FormItem/FormItem.stories.js | 6 +- src/components/FormItem/FormItem.svelte | 16 +- src/components/FormItem/index.js | 4 +- .../FormLabel/FormLabel.Story.svelte | 6 +- src/components/FormLabel/FormLabel.stories.js | 8 +- src/components/FormLabel/FormLabel.svelte | 15 +- src/components/FormLabel/index.js | 4 +- src/components/Icon/Icon.Skeleton.svelte | 12 +- src/components/Icon/Icon.Story.svelte | 12 +- src/components/Icon/Icon.stories.js | 24 +- src/components/Icon/Icon.svelte | 27 +- src/components/Icon/index.js | 6 +- .../InlineLoading/InlineLoading.Story.svelte | 18 +- .../InlineLoading/InlineLoading.stories.js | 30 +- .../InlineLoading/InlineLoading.svelte | 35 +-- .../InlineLoading/InlineLoading.test.js | 28 +- src/components/InlineLoading/index.js | 4 +- src/components/Link/Link.Story.svelte | 9 +- src/components/Link/Link.stories.js | 14 +- src/components/Link/Link.svelte | 31 +-- src/components/Link/Link.test.js | 20 +- src/components/Link/index.js | 4 +- src/components/ListBox/ListBox.svelte | 33 +-- src/components/ListBox/ListBoxField.svelte | 37 ++- src/components/ListBox/ListBoxMenu.svelte | 13 +- src/components/ListBox/ListBoxMenuIcon.svelte | 21 +- src/components/ListBox/ListBoxMenuItem.svelte | 13 +- .../ListBox/ListBoxSelection.svelte | 50 ++-- src/components/ListBox/index.js | 14 +- src/components/ListItem/ListItem.svelte | 16 +- src/components/ListItem/index.js | 4 +- src/components/Loading/Loading.Story.svelte | 4 +- src/components/Loading/Loading.stories.js | 16 +- src/components/Loading/Loading.svelte | 65 +++-- src/components/Loading/Loading.test.js | 24 +- src/components/Loading/index.js | 4 +- src/components/Modal/Modal.Story.svelte | 60 ++-- src/components/Modal/Modal.stories.js | 63 +++-- src/components/Modal/Modal.svelte | 99 +++---- src/components/Modal/index.js | 4 +- .../MultiSelect/MultiSelect.Story.svelte | 27 +- .../MultiSelect/MultiSelect.stories.js | 53 ++-- src/components/MultiSelect/MultiSelect.svelte | 128 +++++---- src/components/MultiSelect/index.js | 4 +- .../Notification/InlineNotification.svelte | 48 ++-- .../Notification/Notification.Story.svelte | 21 +- .../Notification/Notification.stories.js | 62 +++-- .../NotificationActionButton.svelte | 13 +- .../Notification/NotificationButton.svelte | 30 +- .../Notification/NotificationIcon.svelte | 25 +- .../NotificationTextDetails.svelte | 24 +- .../Notification/ToastNotification.svelte | 53 ++-- src/components/Notification/index.js | 12 +- .../NumberInput/NumberInput.Skeleton.svelte | 17 +- .../NumberInput/NumberInput.Story.svelte | 8 +- .../NumberInput/NumberInput.stories.js | 46 ++-- src/components/NumberInput/NumberInput.svelte | 115 +++++--- src/components/NumberInput/index.js | 6 +- .../OrderedList/OrderedList.Story.svelte | 6 +- .../OrderedList/OrderedList.stories.js | 8 +- src/components/OrderedList/OrderedList.svelte | 12 +- src/components/OrderedList/index.js | 4 +- .../OverflowMenu/OverflowMenu.Story.svelte | 38 ++- .../OverflowMenu/OverflowMenu.stories.js | 77 +++--- .../OverflowMenu/OverflowMenu.svelte | 94 ++++--- .../OverflowMenu/OverflowMenuItem.svelte | 38 ++- src/components/OverflowMenu/formatStyle.js | 8 +- src/components/OverflowMenu/index.js | 6 +- .../Pagination/Pagination.Skeleton.svelte | 18 +- .../Pagination/Pagination.Story.svelte | 6 +- .../Pagination/Pagination.stories.js | 84 ++++-- src/components/Pagination/Pagination.svelte | 77 +++--- src/components/Pagination/index.js | 6 +- .../ProgressIndicator.Skeleton.svelte | 25 +- .../ProgressIndicator.Story.svelte | 14 +- .../ProgressIndicator.stories.js | 26 +- .../ProgressIndicator.svelte | 26 +- .../ProgressIndicator/ProgressStep.svelte | 49 ++-- src/components/ProgressIndicator/index.js | 8 +- .../RadioButton/RadioButton.Skeleton.svelte | 18 +- .../RadioButton/RadioButton.Story.svelte | 10 +- .../RadioButton/RadioButton.stories.js | 28 +- src/components/RadioButton/RadioButton.svelte | 50 ++-- src/components/RadioButton/index.js | 6 +- .../RadioButtonGroup.Story.svelte | 10 +- .../RadioButtonGroup.stories.js | 56 ++-- .../RadioButtonGroup/RadioButtonGroup.svelte | 33 +-- src/components/RadioButtonGroup/index.js | 4 +- src/components/Search/Search.Skeleton.svelte | 17 +- src/components/Search/Search.Story.svelte | 12 +- src/components/Search/Search.stories.js | 30 +- src/components/Search/Search.svelte | 77 +++--- src/components/Search/index.js | 6 +- src/components/Select/Select.Skeleton.svelte | 19 +- src/components/Select/Select.Story.svelte | 26 +- src/components/Select/Select.stories.js | 44 +-- src/components/Select/Select.svelte | 106 +++---- src/components/Select/SelectItem.svelte | 29 +- src/components/Select/SelectItemGroup.svelte | 9 +- src/components/Select/index.js | 10 +- .../SkeletonPlaceholder.Story.svelte | 4 +- .../SkeletonPlaceholder.stories.js | 16 +- .../SkeletonPlaceholder.svelte | 14 +- src/components/SkeletonPlaceholder/index.js | 4 +- .../SkeletonText/SkeletonText.Story.svelte | 4 +- .../SkeletonText/SkeletonText.stories.js | 22 +- .../SkeletonText/SkeletonText.svelte | 36 ++- src/components/SkeletonText/index.js | 4 +- src/components/Slider/Slider.Skeleton.svelte | 29 +- src/components/Slider/Slider.Story.svelte | 8 +- src/components/Slider/Slider.stories.js | 49 ++-- src/components/Slider/Slider.svelte | 76 ++--- src/components/Slider/index.js | 6 +- .../StructuredList.Skeleton.svelte | 37 ++- .../StructuredList.Story.svelte | 54 ++-- .../StructuredList/StructuredList.stories.js | 10 +- .../StructuredList/StructuredList.svelte | 27 +- .../StructuredList/StructuredListBody.svelte | 14 +- .../StructuredList/StructuredListCell.svelte | 13 +- .../StructuredList/StructuredListHead.svelte | 14 +- .../StructuredList/StructuredListInput.svelte | 30 +- .../StructuredList/StructuredListRow.svelte | 24 +- src/components/StructuredList/index.js | 17 +- src/components/Tabs/Tab.svelte | 41 ++- src/components/Tabs/TabContent.svelte | 18 +- src/components/Tabs/Tabs.Story.svelte | 12 +- src/components/Tabs/Tabs.stories.js | 55 ++-- src/components/Tabs/Tabs.svelte | 41 +-- src/components/Tabs/TabsSkeleton.svelte | 25 +- src/components/Tabs/index.js | 10 +- src/components/Tag/Tag.svelte | 6 +- .../TextArea/TextArea.Skeleton.svelte | 17 +- src/components/TextArea/TextArea.Story.svelte | 13 +- src/components/TextArea/TextArea.stories.js | 36 +-- src/components/TextArea/TextArea.svelte | 70 +++-- src/components/TextArea/index.js | 6 +- src/components/TextInput/PasswordInput.svelte | 96 ++++--- .../TextInput/TextInput.Skeleton.svelte | 17 +- .../TextInput/TextInput.Story.svelte | 21 +- src/components/TextInput/TextInput.stories.js | 120 ++++---- src/components/TextInput/TextInput.svelte | 71 ++--- src/components/TextInput/index.js | 8 +- src/components/Tile/ClickableTile.svelte | 19 +- src/components/Tile/ExpandableTile.svelte | 68 ++--- src/components/Tile/RadioTile.svelte | 53 ++-- src/components/Tile/SelectableTile.svelte | 49 ++-- src/components/Tile/Tile.Story.svelte | 45 +-- src/components/Tile/Tile.stories.js | 54 ++-- src/components/Tile/Tile.svelte | 12 +- src/components/Tile/TileGroup.svelte | 23 +- src/components/Tile/index.js | 14 +- .../TimePicker/TimePicker.Story.svelte | 29 +- .../TimePicker/TimePicker.stories.js | 64 +++-- src/components/TimePicker/TimePicker.svelte | 53 ++-- .../TimePicker/TimePickerSelect.svelte | 44 +-- src/components/TimePicker/index.js | 6 +- src/components/Toggle/Toggle.Skeleton.svelte | 36 ++- src/components/Toggle/Toggle.Story.svelte | 13 +- src/components/Toggle/Toggle.stories.js | 44 +-- src/components/Toggle/Toggle.svelte | 48 ++-- src/components/Toggle/index.js | 6 +- .../ToggleSmall/ToggleSmall.Skeleton.svelte | 40 ++- .../ToggleSmall/ToggleSmall.Story.svelte | 19 +- .../ToggleSmall/ToggleSmall.stories.js | 44 +-- src/components/ToggleSmall/ToggleSmall.svelte | 61 ++-- src/components/ToggleSmall/index.js | 6 +- src/components/Tooltip/Tooltip.Story.svelte | 33 ++- src/components/Tooltip/Tooltip.stories.js | 58 ++-- src/components/Tooltip/Tooltip.svelte | 104 ++++--- src/components/Tooltip/index.js | 4 +- .../TooltipDefinition.Story.svelte | 6 +- .../TooltipDefinition.stories.js | 34 +-- .../TooltipDefinition.svelte | 51 +++- src/components/TooltipDefinition/index.js | 4 +- .../TooltipIcon/TooltipIcon.Story.svelte | 6 +- .../TooltipIcon/TooltipIcon.stories.js | 28 +- src/components/TooltipIcon/TooltipIcon.svelte | 41 ++- src/components/TooltipIcon/index.js | 4 +- src/components/UIShell/FormTest.svelte | 21 +- src/components/UIShell/SkipToContent.svelte | 2 +- .../UnorderedList/UnorderedList.Story.svelte | 6 +- .../UnorderedList/UnorderedList.stories.js | 8 +- .../UnorderedList/UnorderedList.svelte | 12 +- src/components/UnorderedList/index.js | 4 +- src/index.js | 260 +++++------------- src/internal/ui/Layout.svelte | 13 +- src/lib/__tests__/css.spec.js | 9 - src/lib/__tests__/fill-array.spec.js | 7 - src/lib/css.js | 16 -- src/lib/fill-array.js | 5 - src/lib/index.js | 4 +- yarn.lock | 8 +- 288 files changed, 4681 insertions(+), 4498 deletions(-) create mode 100644 .storybook/preview-body.html delete mode 100644 src/lib/__tests__/css.spec.js delete mode 100644 src/lib/__tests__/fill-array.spec.js delete mode 100644 src/lib/css.js delete mode 100644 src/lib/fill-array.js diff --git a/.storybook/preview-body.html b/.storybook/preview-body.html new file mode 100644 index 00000000..15966e2e --- /dev/null +++ b/.storybook/preview-body.html @@ -0,0 +1,8 @@ + diff --git a/.storybook/preview-head.html b/.storybook/preview-head.html index 74f76ab2..a5204e1e 100644 --- a/.storybook/preview-head.html +++ b/.storybook/preview-head.html @@ -1 +1,4 @@ - \ No newline at end of file + diff --git a/package.json b/package.json index 2c6fadd2..6793ca6e 100644 --- a/package.json +++ b/package.json @@ -16,7 +16,7 @@ }, "sideEffects": false, "dependencies": { - "carbon-icons-svelte": "10.9.3", + "carbon-icons-svelte": "^10.13.0", "flatpickr": "4.6.3" }, "devDependencies": { diff --git a/src/components/Accordion/Accordion.Skeleton.svelte b/src/components/Accordion/Accordion.Skeleton.svelte index 73b2def3..436d57ea 100644 --- a/src/components/Accordion/Accordion.Skeleton.svelte +++ b/src/components/Accordion/Accordion.Skeleton.svelte @@ -1,40 +1,39 @@