feat(statistic): new component #3329
Conversation
liweijie0812
requested review from
PengYYYYY,
chaishi,
xiaosansiji,
honkinglin,
LeeJim and
uyarn
as code owners
|
|
|
|
1.标题文案token使用错误，应该用
|
|
1.数据的 2.unit的字符用错token了，这里应该是 3.组合使用，箭头没有居中对齐 4.数值动画的后缀大小要用token， |
DONE |
| }; | ||
|
|
||
| const formatValue = computed(() => { | ||
| let _value: number | undefined | string = innerValue.value; |
There was a problem hiding this comment.
eslint 开启no-underscore-dangle，禁止下划线开头
检查结果
|
|
||
| onMounted(() => props.animation && props.animationStart && start()); | ||
|
|
||
| expose({ start }); |
There was a problem hiding this comment.
这里抛出了组件实例方法，但是 API 文档没有呈现，用户无法感知。
API 平台已录入该方法，辛苦使用 npm run api:docs Statistic "VueNext(PC)" finalProject 更新 API 文档
| const trendIcons = { | ||
| increase: <ArrowTriangleUpFilledIcon />, | ||
| decrease: <ArrowTriangleDownFilledIcon />, | ||
| }; |
There was a problem hiding this comment.
常量 trendIcons 建议定义在组件最外面，而非 render 函数里，可避免每次渲染是计算时重复创建对象。一个数字可能没关系，但如果放在 Table 数据表中，就会变很多。
const trendIcons = {
increase: ArrowTriangleUpFilledIcon,
decrease: ArrowTriangleDownFilledIcon,
};
| innerValue.value = to; | ||
| }, | ||
| }); | ||
| (tween.value as any)?.start(); |
There was a problem hiding this comment.
这里使用了 as any，需要再次确认下具体是哪部分代码导致了 TS 类型丢失
🤔 这个 PR 的性质是？
🔗 相关 Issue
迁移vue2 statistic PR Tencent/tdesign-vue#2397
💡 需求背景和解决方案
📝 更新日志
feat(statistic): 新增
Statistic统计数值组件本条 PR 不需要纳入 Changelog
☑️ 请求合并前的自查清单