ant-design / pro-components

🏆 Use Ant Design like a Pro!
https://pro-components.antdigital.dev
MIT License
4.04k stars 1.29k forks source link

feat(core):添加 ListToolBar 列表工具栏子组件 #373

Closed rdmclin2 closed 3 years ago

rdmclin2 commented 3 years ago
  1. 添加 ListToolBar 组件 到 ProUtils 中
  2. 添加 ListToolBar demos
  3. 更新 snapshot.
github-actions[bot] commented 3 years ago

🎊 PR Preview f619cea55b229b241f74878c0fdd82f5c7e67c56 has been successfully built and deployed to https://ant-design-pro-components-preview-pr-373.surge.sh

:clock1: Build time: 1039.962s

🤖 By surge-preview

chenshuai2144 commented 3 years ago

做到 List 里面了呀,table怎么用呢?

github-actions[bot] commented 3 years ago

size-limit report

Path Size
packages/card/lib/cardLoading.js, packages/card/lib/index.js 4.04 KB (-0.44% 🔽)
packages/card/es/cardLoading.js, packages/card/es/index.js 3.8 KB (-0.52% 🔽)
packages/descriptions/lib/index.js, packages/descriptions/lib/useFetchData.js 5.61 KB (0%)
packages/descriptions/es/index.js, packages/descriptions/es/useFetchData.js 5.1 KB (0%)
packages/field/lib/index.js, packages/field/lib/components/Code/index.js, packages/field/lib/components/DatePicker/index.js, packages/field/lib/components/Digit/index.js, packages/field/lib/components/IndexColumn/index.js, packages/field/lib/components/Money/index.js, packages/field/lib/components/Options/index.js, packages/field/lib/components/Percent/index.js, packages/field/lib/components/Percent/util.js, packages/field/lib/components/Progress/index.js, packages/field/lib/components/RangePicker/index.js, packages/field/lib/components/Select/index.js, packages/field/lib/components/Status/index.js, packages/field/lib/components/Text/index.js, packages/field/lib/components/TextArea/index.js, packages/field/lib/components/TimePicker/index.js, packages/field/lib/components/Select/LightSelect/index.js 31.95 KB (0%)
packages/field/es/index.js, packages/field/es/components/Code/index.js, packages/field/es/components/DatePicker/index.js, packages/field/es/components/Digit/index.js, packages/field/es/components/IndexColumn/index.js, packages/field/es/components/Money/index.js, packages/field/es/components/Options/index.js, packages/field/es/components/Percent/index.js, packages/field/es/components/Percent/util.js, packages/field/es/components/Progress/index.js, packages/field/es/components/RangePicker/index.js, packages/field/es/components/Select/index.js, packages/field/es/components/Status/index.js, packages/field/es/components/Text/index.js, packages/field/es/components/TextArea/index.js, packages/field/es/components/TimePicker/index.js, packages/field/es/components/Select/LightSelect/index.js 26.59 KB (0%)
packages/form/lib/FieldContext.js, packages/form/lib/index.js, packages/form/lib/BaseForm/index.js, packages/form/lib/BaseForm/LightWrapper/index.js, packages/form/lib/components/Checkbox/index.js, packages/form/lib/components/DatePicker/DatePicker.js, packages/form/lib/components/DatePicker/MonthPicker.js, packages/form/lib/components/DatePicker/QuarterPicker.js, packages/form/lib/components/DatePicker/WeekPicker.js, packages/form/lib/components/DatePicker/YearPicker.js, packages/form/lib/components/DatePicker/index.js, packages/form/lib/components/DateRangePicker/index.js, packages/form/lib/components/DateTimePicker/index.js, packages/form/lib/components/DateTimeRangePicker/index.js, packages/form/lib/components/Digit/index.js, packages/form/lib/components/Field/index.js, packages/form/lib/components/Group/index.js, packages/form/lib/components/Radio/index.js, packages/form/lib/components/Rate/index.js, packages/form/lib/components/Select/index.js, packages/form/lib/components/Slider/index.js, packages/form/lib/components/Submitter/index.js, packages/form/lib/components/Switch/index.js, packages/form/lib/components/Text/index.js, packages/form/lib/components/TextArea/index.js, packages/form/lib/components/TimePicker/index.js, packages/form/lib/components/UploadButton/index.js, packages/form/lib/components/UploadDragger/index.js, packages/form/lib/layouts/LightFilter/index.js, packages/form/lib/layouts/ProForm/index.js, packages/form/lib/layouts/QueryFilter/Actions.js, packages/form/lib/layouts/QueryFilter/index.js, packages/form/lib/layouts/StepsFrom/StepFrom.js, packages/form/lib/layouts/StepsFrom/index.js 42.74 KB (0%)
packages/form/es/FieldContext.js, packages/form/es/index.js, packages/form/es/BaseForm/index.js, packages/form/es/BaseForm/LightWrapper/index.js, packages/form/es/components/Checkbox/index.js, packages/form/es/components/DatePicker/DatePicker.js, packages/form/es/components/DatePicker/MonthPicker.js, packages/form/es/components/DatePicker/QuarterPicker.js, packages/form/es/components/DatePicker/WeekPicker.js, packages/form/es/components/DatePicker/YearPicker.js, packages/form/es/components/DatePicker/index.js, packages/form/es/components/DateRangePicker/index.js, packages/form/es/components/DateTimePicker/index.js, packages/form/es/components/DateTimeRangePicker/index.js, packages/form/es/components/Digit/index.js, packages/form/es/components/Field/index.js, packages/form/es/components/Group/index.js, packages/form/es/components/Radio/index.js, packages/form/es/components/Rate/index.js, packages/form/es/components/Select/index.js, packages/form/es/components/Slider/index.js, packages/form/es/components/Submitter/index.js, packages/form/es/components/Switch/index.js, packages/form/es/components/Text/index.js, packages/form/es/components/TextArea/index.js, packages/form/es/components/TimePicker/index.js, packages/form/es/components/UploadButton/index.js, packages/form/es/components/UploadDragger/index.js, packages/form/es/layouts/LightFilter/index.js, packages/form/es/layouts/ProForm/index.js, packages/form/es/layouts/QueryFilter/Actions.js, packages/form/es/layouts/QueryFilter/index.js, packages/form/es/layouts/StepsFrom/StepFrom.js, packages/form/es/layouts/StepsFrom/index.js 35.8 KB (0%)
packages/layout/lib/BasicLayout.js, packages/layout/lib/Footer.js, packages/layout/lib/Header.js, packages/layout/lib/RouteContext.js, packages/layout/lib/WrapContent.js, packages/layout/lib/defaultSettings.js, packages/layout/lib/getPageTitle.js, packages/layout/lib/index.js, packages/layout/lib/typings.js, packages/layout/lib/GlobalFooter/index.js, packages/layout/lib/FooterToolbar/index.js, packages/layout/lib/GlobalHeader/index.js, packages/layout/lib/GridContent/index.js, packages/layout/lib/PageContainer/index.js, packages/layout/lib/PageLoading/index.js, packages/layout/lib/SettingDrawer/BlockCheckbox.js, packages/layout/lib/SettingDrawer/LayoutChange.js, packages/layout/lib/SettingDrawer/RegionalChange.js, packages/layout/lib/SettingDrawer/ThemeColor.js, packages/layout/lib/SettingDrawer/index.js, packages/layout/lib/SiderMenu/BaseMenu.js, packages/layout/lib/SiderMenu/Counter.js, packages/layout/lib/SiderMenu/SiderMenu.js, packages/layout/lib/SiderMenu/SiderMenuUtils.js, packages/layout/lib/SiderMenu/index.js, packages/layout/lib/TopNavHeader/index.js, packages/layout/lib/locales/en-US.js, packages/layout/lib/locales/index.js, packages/layout/lib/locales/it-IT.js, packages/layout/lib/locales/zh-CN.js, packages/layout/lib/locales/zh-TW.js, packages/layout/lib/utils/compatibleLayout.js, packages/layout/lib/utils/getBreadcrumbProps.js, packages/layout/lib/utils/getMenuData.js, packages/layout/lib/utils/pathTools.js, packages/layout/lib/utils/utils.js, packages/layout/lib/locales/en-US/settingDrawer.js, packages/layout/lib/locales/it-IT/settingDrawer.js, packages/layout/lib/locales/zh-CN/settingDrawer.js, packages/layout/lib/locales/zh-TW/settingDrawer.js 0 B (-100%)
packages/layout/es/BasicLayout.js, packages/layout/es/Footer.js, packages/layout/es/Header.js, packages/layout/es/RouteContext.js, packages/layout/es/WrapContent.js, packages/layout/es/defaultSettings.js, packages/layout/es/getPageTitle.js, packages/layout/es/index.js, packages/layout/es/typings.js, packages/layout/es/FooterToolbar/index.js, packages/layout/es/GlobalFooter/index.js, packages/layout/es/GlobalHeader/index.js, packages/layout/es/GridContent/index.js, packages/layout/es/PageContainer/index.js, packages/layout/es/PageLoading/index.js, packages/layout/es/SettingDrawer/BlockCheckbox.js, packages/layout/es/SettingDrawer/LayoutChange.js, packages/layout/es/SettingDrawer/RegionalChange.js, packages/layout/es/SettingDrawer/ThemeColor.js, packages/layout/es/SettingDrawer/index.js, packages/layout/es/SiderMenu/BaseMenu.js, packages/layout/es/SiderMenu/Counter.js, packages/layout/es/SiderMenu/SiderMenu.js, packages/layout/es/SiderMenu/SiderMenuUtils.js, packages/layout/es/SiderMenu/index.js, packages/layout/es/TopNavHeader/index.js, packages/layout/es/locales/en-US.js, packages/layout/es/locales/index.js, packages/layout/es/locales/it-IT.js, packages/layout/es/locales/zh-CN.js, packages/layout/es/locales/zh-TW.js, packages/layout/es/utils/compatibleLayout.js, packages/layout/es/utils/getBreadcrumbProps.js, packages/layout/es/utils/getMenuData.js, packages/layout/es/utils/pathTools.js, packages/layout/es/utils/utils.js, packages/layout/es/locales/en-US/settingDrawer.js, packages/layout/es/locales/it-IT/settingDrawer.js, packages/layout/es/locales/zh-CN/settingDrawer.js, packages/layout/es/locales/zh-TW/settingDrawer.js 45.18 KB (0%)
packages/list/lib/Item.js, packages/list/lib/index.js, packages/list/lib/hooks/useLazyKVMap.js, packages/list/lib/hooks/usePagination.js, packages/list/lib/hooks/useSelection.js, packages/list/lib/toolBar/index.js, packages/list/lib/util/getPrefixCls.js 0 B (-100%)
packages/list/es/Item.js, packages/list/es/index.js, packages/list/es/hooks/useLazyKVMap.js, packages/list/es/hooks/usePagination.js, packages/list/es/hooks/useSelection.js, packages/list/es/toolBar/index.js, packages/list/es/util/getPrefixCls.js 0 B (-100%)
packages/provider/lib/index.js, packages/provider/lib/locale/en_US.js, packages/provider/lib/locale/es_ES.js, packages/provider/lib/locale/fr_FR.js, packages/provider/lib/locale/it_IT.js, packages/provider/lib/locale/ja_JP.js, packages/provider/lib/locale/ms_MY.js, packages/provider/lib/locale/pt_BR.js, packages/provider/lib/locale/ru_RU.js, packages/provider/lib/locale/vi_VN.js, packages/provider/lib/locale/zh_CN.js, packages/provider/lib/locale/zh_TW.js 9.66 KB (0%)
packages/provider/es/index.js, packages/provider/es/locale/en_US.js, packages/provider/es/locale/es_ES.js, packages/provider/es/locale/fr_FR.js, packages/provider/es/locale/it_IT.js, packages/provider/es/locale/ja_JP.js, packages/provider/es/locale/ms_MY.js, packages/provider/es/locale/pt_BR.js, packages/provider/es/locale/ru_RU.js, packages/provider/es/locale/vi_VN.js, packages/provider/es/locale/zh_CN.js, packages/provider/es/locale/zh_TW.js 8.23 KB (0%)
packages/skeleton/lib/index.js, packages/skeleton/lib/component/Descriptions/index.js, packages/skeleton/lib/component/List/index.js, packages/skeleton/lib/component/Result/index.js 4.95 KB (0%)
packages/skeleton/es/index.js, packages/skeleton/es/component/Descriptions/index.js, packages/skeleton/es/component/List/index.js, packages/skeleton/es/component/Result/index.js 3.96 KB (0%)
packages/table/lib/Table.js, packages/table/lib/container.js, packages/table/lib/defaultRender.js, packages/table/lib/index.js, packages/table/lib/useFetchData.js, packages/table/lib/utils.js, packages/table/lib/Form/index.js, packages/table/lib/component/ErrorBoundary.js, packages/table/lib/component/Alert/index.js, packages/table/lib/component/ColumnSetting/DndItem.js, packages/table/lib/component/ColumnSetting/DragIcon.js, packages/table/lib/component/ColumnSetting/index.js, packages/table/lib/component/Dropdown/index.js, packages/table/lib/component/ToolBar/DensityIcon.js, packages/table/lib/component/ToolBar/FullscreenIcon.js, packages/table/lib/component/ToolBar/index.js 36.95 KB (-1.96% 🔽)
packages/table/es/Table.js, packages/table/es/container.js, packages/table/es/defaultRender.js, packages/table/es/index.js, packages/table/es/useFetchData.js, packages/table/es/utils.js, packages/table/es/Form/index.js, packages/table/es/component/ErrorBoundary.js, packages/table/es/component/Alert/index.js, packages/table/es/component/ColumnSetting/DndItem.js, packages/table/es/component/ColumnSetting/DragIcon.js, packages/table/es/component/ColumnSetting/index.js, packages/table/es/component/Dropdown/index.js, packages/table/es/component/ToolBar/DensityIcon.js, packages/table/es/component/ToolBar/FullscreenIcon.js, packages/table/es/component/ToolBar/index.js 32.23 KB (-1.21% 🔽)
packages/layout/lib/BasicLayout.js, packages/layout/lib/Footer.js, packages/layout/lib/Header.js, packages/layout/lib/RouteContext.js, packages/layout/lib/WrapContent.js, packages/layout/lib/defaultSettings.js, packages/layout/lib/getPageTitle.js, packages/layout/lib/index.js, packages/layout/lib/typings.js, packages/layout/lib/FooterToolbar/index.js, packages/layout/lib/GlobalFooter/index.js, packages/layout/lib/GlobalHeader/index.js, packages/layout/lib/GridContent/index.js, packages/layout/lib/PageContainer/index.js, packages/layout/lib/PageLoading/index.js, packages/layout/lib/SettingDrawer/BlockCheckbox.js, packages/layout/lib/SettingDrawer/LayoutChange.js, packages/layout/lib/SettingDrawer/RegionalChange.js, packages/layout/lib/SettingDrawer/ThemeColor.js, packages/layout/lib/SettingDrawer/index.js, packages/layout/lib/SiderMenu/BaseMenu.js, packages/layout/lib/SiderMenu/Counter.js, packages/layout/lib/SiderMenu/SiderMenu.js, packages/layout/lib/SiderMenu/SiderMenuUtils.js, packages/layout/lib/SiderMenu/index.js, packages/layout/lib/TopNavHeader/index.js, packages/layout/lib/locales/en-US.js, packages/layout/lib/locales/index.js, packages/layout/lib/locales/it-IT.js, packages/layout/lib/locales/zh-CN.js, packages/layout/lib/locales/zh-TW.js, packages/layout/lib/utils/compatibleLayout.js, packages/layout/lib/utils/getBreadcrumbProps.js, packages/layout/lib/utils/getMenuData.js, packages/layout/lib/utils/pathTools.js, packages/layout/lib/utils/utils.js, packages/layout/lib/locales/en-US/settingDrawer.js, packages/layout/lib/locales/it-IT/settingDrawer.js, packages/layout/lib/locales/zh-CN/settingDrawer.js, packages/layout/lib/locales/zh-TW/settingDrawer.js 53.05 KB (+100% 🔺)
packages/list/lib/Item.js, packages/list/lib/index.js, packages/list/lib/hooks/useLazyKVMap.js, packages/list/lib/hooks/usePagination.js, packages/list/lib/hooks/useSelection.js, packages/list/lib/util/getPrefixCls.js 14.14 KB (+100% 🔺)
packages/list/es/Item.js, packages/list/es/index.js, packages/list/es/hooks/useLazyKVMap.js, packages/list/es/hooks/usePagination.js, packages/list/es/hooks/useSelection.js, packages/list/es/util/getPrefixCls.js 12.55 KB (+100% 🔺)
codecov[bot] commented 3 years ago

Codecov Report

Merging #373 into master will increase coverage by 3.73%. The diff coverage is 93.13%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #373      +/-   ##
==========================================
+ Coverage   86.10%   89.84%   +3.73%     
==========================================
  Files         131      133       +2     
  Lines        3591     3623      +32     
  Branches     1291     1304      +13     
==========================================
+ Hits         3092     3255     +163     
+ Misses        494      365     -129     
+ Partials        5        3       -2     
Impacted Files Coverage Δ
packages/descriptions/src/index.tsx 92.85% <0.00%> (ø)
packages/field/src/index.tsx 97.14% <ø> (ø)
...ckages/layout/src/SettingDrawer/RegionalChange.tsx 100.00% <ø> (+12.50%) :arrow_up:
packages/layout/src/locales/index.ts 100.00% <ø> (+7.69%) :arrow_up:
packages/table/src/component/Alert/index.tsx 100.00% <ø> (+22.72%) :arrow_up:
...ckages/table/src/component/ColumnSetting/index.tsx 83.83% <ø> (-0.17%) :arrow_down:
packages/table/src/Form/index.tsx 96.74% <71.42%> (+8.55%) :arrow_up:
...ackages/utils/src/components/ListToolBar/index.tsx 94.11% <94.11%> (ø)
...es/utils/src/components/ListToolBar/HeaderMenu.tsx 94.44% <94.44%> (ø)
packages/layout/src/SettingDrawer/LayoutChange.tsx 88.23% <100.00%> (+19.48%) :arrow_up:
... and 18 more

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 5e9880a...0dc0864. Read the comment docs.

rdmclin2 commented 3 years ago

正好你不在,没时间跟你讨论放哪里好呢 @chenshuai2144

rdmclin2 commented 3 years ago

加到 ProUtils 中了 @chenshuai2144

chenshuai2144 commented 3 years ago

能把 demo 做到各个组件里面吗? utils里面只做测试就好了

rdmclin2 commented 3 years ago

能把 demo 做到各个组件里面吗? utils里面只做测试就好了

组件也可以单独使用,感觉还是放 utils 的这个位置比较好,放在哪个组件里面都不合适的。除了ListToolBar,还有 BatchOperationBar。

rdmclin2 commented 3 years ago

ProTable 和 List 让直接内置吧

我像先把 ListToolBar 和 BatchOperationBar 都先迁移过来,然后再一起考虑内置到 ProTable 和 ProList 中

chenshuai2144 commented 3 years ago

总要改的,如果 table 和 list的功能都不能支持,那用起来一定很难受

api 能修改的早点改掉,不然以后改不动

rdmclin2 commented 3 years ago

好的,那我尝试迁移一下

rdmclin2 commented 3 years ago

cc @chenshuai2144 @yutingzhao1991 ,再看一下

yutingzhao1991 commented 3 years ago

CI 要修复下。

yutingzhao1991 commented 3 years ago

有冲突了。

chenshuai2144 commented 3 years ago

ci 全没了

rdmclin2 commented 3 years ago

代码一更新就冲突

lgtm-com[bot] commented 3 years ago

This pull request fixes 1 alert when merging b7ab347abd862bcbdcc7341ae79d6089094d7ca9 into 5e9880ab01da5a71beb573e9cd1c71749283dd28 - view on LGTM.com

fixed alerts:

lgtm-com[bot] commented 3 years ago

This pull request fixes 1 alert when merging 0dc08643a832c3abe595ef789a11c0e9a0bc55af into 5e9880ab01da5a71beb573e9cd1c71749283dd28 - view on LGTM.com

fixed alerts: