vue-devui icon indicating copy to clipboard operation
vue-devui copied to clipboard

refactor(List): 重构List组件

Open LadyChatterleyLover opened this issue 3 years ago • 20 comments

重构List组件

LadyChatterleyLover avatar Aug 23 '22 01:08 LadyChatterleyLover

@LadyChatterleyLover build-preview任务不通过

kagol avatar Aug 23 '22 02:08 kagol

@kagol 请问这个构建是什么问题呢?报错了。

LadyChatterleyLover avatar Aug 23 '22 03:08 LadyChatterleyLover

@kagol 请问这个构建是什么问题呢?报错了。

你尝试本地执行下:pnpm scripts run builds,看下是否成功,如果不成功,看下报什么错 @LadyChatterleyLover

kagol avatar Aug 23 '22 10:08 kagol

好的,谢谢 。

发自我的iPhone

------------------ 原始邮件 ------------------ 发件人: Kagol @.> 发送时间: 2022年8月23日 18:00 收件人: DevCloudFE/vue-devui @.> 抄送: luopei @.>, Mention @.> 主题: Re: [DevCloudFE/vue-devui] refactor(List): 重构List组件 (PR #1262)

@kagol 请问这个构建是什么问题呢?报错了。

你尝试本地执行下:pnpm scripts run buils,看下是否成功,如果不成功,看下报什么错 @LadyChatterleyLover

— Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you were mentioned.Message ID: @.***>

LadyChatterleyLover avatar Aug 23 '22 10:08 LadyChatterleyLover

@kagol 不行,本地没那条命令

LadyChatterleyLover avatar Aug 23 '22 11:08 LadyChatterleyLover

@kagol 不行,本地没那条命令

@LadyChatterleyLover 那你尝试使用以下命令同步下上游最新代码:

git pull upstream dev

kagol avatar Aug 24 '22 01:08 kagol

好的,我试一下

------------------ 原始邮件 ------------------ 发件人: "DevCloudFE/vue-devui" @.>; 发送时间: 2022年8月24日(星期三) 上午9:32 @.>; @.@.>; 主题: Re: [DevCloudFE/vue-devui] refactor(List): 重构List组件 (PR #1262)

@kagol 不行,本地没那条命令

那你尝试使用以下命令同步下上游最新代码: git pull upstream dev
— Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you were mentioned.Message ID: @.***>

LadyChatterleyLover avatar Aug 24 '22 01:08 LadyChatterleyLover

PR preview has been successfully built and deployed to https://vue-devui-pr-1262.surge.sh.

github-actions[bot] avatar Aug 24 '22 02:08 github-actions[bot]

@LadyChatterleyLover 组件未显示:https://vue-devui-pr-1262.surge.sh/components/list/

image

kagol avatar Aug 24 '22 11:08 kagol

PR preview has been successfully built and deployed to https://vue-devui-pr-1262.surge.sh.

github-actions[bot] avatar Aug 24 '22 12:08 github-actions[bot]

PR preview has been successfully built and deployed to https://vue-devui-pr-1262.surge.sh.

github-actions[bot] avatar Aug 24 '22 12:08 github-actions[bot]

PR preview has been successfully built and deployed to https://vue-devui-pr-1262.surge.sh.

github-actions[bot] avatar Aug 24 '22 12:08 github-actions[bot]

@kagol 这个报错我没看出来是什么。。。

LadyChatterleyLover avatar Aug 24 '22 12:08 LadyChatterleyLover

@kagol 这个报错我没看出来是什么。。。

@LadyChatterleyLover 应该是使用了Loading指令和Pagination组件,但是没有引入导致的,然后loading指令应该是v-loading,不是v-d-loading

import { Pagination, PaginationProps } from '../../pagination';
import { LoadingDirective } from '../../loading';

export default defineComponent({
  name: 'DList',
  components: {
    'd-pagination': Pagination,
  },
  directives: {
    Loading: LoadingDirective,
  },
  ...
});

kagol avatar Aug 24 '22 13:08 kagol

@LadyChatterleyLover 另外分页功能不建议集成在List里面,用户可以自己使用List和Pagination组合出分页的效果。

kagol avatar Aug 24 '22 14:08 kagol

好的,明天改一下 。

发自我的iPhone

------------------ 原始邮件 ------------------ 发件人: Kagol @.> 发送时间: 2022年8月24日 21:59 收件人: DevCloudFE/vue-devui @.> 抄送: luopei @.>, Mention @.> 主题: Re: [DevCloudFE/vue-devui] refactor(List): 重构List组件 (PR #1262)

@kagol 这个报错我没看出来是什么。。。

@LadyChatterleyLover 应该是引入了Loading指令和Pagination组件,但是没有引入导致的,然后loading指令应该是v-loading,不是v-d-loading。 import { Pagination, PaginationProps } from '../../pagination'; import { LoadingDirective } from '../../loading'; export default defineComponent({ name: 'DList', components: { 'd-pagination': Pagination, }, directives: { Loading: LoadingDirective, }, ... });
— Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you were mentioned.Message ID: @.***>

LadyChatterleyLover avatar Aug 24 '22 14:08 LadyChatterleyLover

PR preview has been successfully built and deployed to https://vue-devui-pr-1262.surge.sh.

github-actions[bot] avatar Aug 25 '22 02:08 github-actions[bot]

@kagol 关于分页,这个我是参考了其他组件库的设计,而且只是给用户提供一个选项,可以使用,也可以不使用,他也可以自行用分页组件进行分页,我觉得这个功能可以保留哈,或者我们可以再讨论一下

LadyChatterleyLover avatar Aug 25 '22 03:08 LadyChatterleyLover

@kagol 关于分页,这个我是参考了其他组件库的设计,而且只是给用户提供一个选项,可以使用,也可以不使用,他也可以自行用分页组件进行分页,我觉得这个功能可以保留哈,或者我们可以再讨论一下

可以的,下次田主大会咱们一起讨论下🤝

kagol avatar Sep 05 '22 13:09 kagol

好的,

发自我的iPhone

------------------ 原始邮件 ------------------ 发件人: Kagol @.> 发送时间: 2022年9月5日 21:15 收件人: DevCloudFE/vue-devui @.> 抄送: luopei @.>, Mention @.> 主题: Re: [DevCloudFE/vue-devui] refactor(List): 重构List组件 (PR #1262)

@kagol 关于分页,这个我是参考了其他组件库的设计,而且只是给用户提供一个选项,可以使用,也可以不使用,他也可以自行用分页组件进行分页,我觉得这个功能可以保留哈,或者我们可以再讨论一下

可以的,下次田主大会咱们一起讨论下🤝

— Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you were mentioned.Message ID: @.***>

LadyChatterleyLover avatar Sep 05 '22 14:09 LadyChatterleyLover