Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Table] ellipsisTitle优先级低于ellipsis,与预期不符 #1404

Closed
cong-min opened this issue Aug 29, 2022 · 13 comments
Closed

[Table] ellipsisTitle优先级低于ellipsis,与预期不符 #1404

cong-min opened this issue Aug 29, 2022 · 13 comments
Assignees
Labels
need more info need more info

Comments

@cong-min
Copy link
Contributor

tdesign-vue 版本

0.46.2

重现链接

No response

重现步骤

image

文档中描述 ellipsisTitle 优先级高于ellipsis (预期),
而实际上当 ellipsis 为 false,ellipsisTitle 为 true 时并不能生效:

{col.ellipsis && ellipsisTitle !== false && ellipsisTitle !== null ? (

期望结果

No response

实际结果

No response

框架版本

No response

浏览器版本

No response

系统版本

No response

Node版本

No response

补充说明

No response

@github-actions
Copy link
Contributor

👋 @cong-min,感谢给 TDesign 提出了 issue。
请根据 issue 模版确保背景信息的完善,我们将调查并尽快回复你。

@github-actions
Copy link
Contributor

♥️ 有劳 @realyuyanan @chaishi @cool-518 尽快确认问题。
确认有效后将下一步计划和可能需要的时间回复给 @cong-min

@chaishi chaishi added the 🙏 help wanted Extra attention is needed label Aug 29, 2022
@github-actions
Copy link
Contributor

任何人都可以处理此问题。
请务必在您的 pull request 中引用此问题。
感谢你的贡献! ✨

@Tomaolala
Copy link
Contributor

认领 @chaishi

@chaishi
Copy link
Collaborator

chaishi commented Sep 5, 2022

Vue2 已合并

issue 先别关,还有 Vue3 和 React,问题相同。需要有缘人处理

@yusongh
Copy link
Contributor

yusongh commented Sep 17, 2022

如果 #1408 的处理是符合预期的,ellipsis的描述应该需要修改,ellipsis配置并不会控制表头的文字是否省略了。

例如:ellipsisTitle不配置,则为默认值undefined。ellipsis配置为true和false表头文字都是省略的

image

但如果是按照当前ellipsis描述才是符合预期的,应该是判断ellipsisTitle是否配置了,没有配置则取ellipsis的值吧

const isEllipsis = (ellipsis: BaseTableCol['ellipsis'], ellipsisTitle: BaseTableCol['ellipsisTitle']) => {
  if (ellipsisTitle === undefined) {
    return ellipsis;
  }

  if (ellipsisTitle !== false && ellipsisTitle !== null) {
    return true;
  }

  return false;
};

@Tomaolala
Copy link
Contributor

如果 #1408 的处理是符合预期的,ellipsis的描述应该需要修改,ellipsis配置并不会控制表头的文字是否省略了。

例如:ellipsisTitle不配置,则为默认值undefined。ellipsis配置为true和false表头文字都是省略的

image

但如果是按照当前ellipsis描述才是符合预期的,应该是判断ellipsisTitle是否配置了,没有配置则取ellipsis的值吧

const isEllipsis = (ellipsis: BaseTableCol['ellipsis'], ellipsisTitle: BaseTableCol['ellipsisTitle']) => {
  if (ellipsisTitle === undefined) {
    return ellipsis;
  }

  if (ellipsisTitle !== false && ellipsisTitle !== null) {
    return true;
  }

  return false;
};

image

如果 #1408 的处理是符合预期的,ellipsis的描述应该需要修改,ellipsis配置并不会控制表头的文字是否省略了。

例如:ellipsisTitle不配置,则为默认值undefined。ellipsis配置为true和false表头文字都是省略的

image

但如果是按照当前ellipsis描述才是符合预期的,应该是判断ellipsisTitle是否配置了,没有配置则取ellipsis的值吧

const isEllipsis = (ellipsis: BaseTableCol['ellipsis'], ellipsisTitle: BaseTableCol['ellipsisTitle']) => {
  if (ellipsisTitle === undefined) {
    return ellipsis;
  }

  if (ellipsisTitle !== false && ellipsisTitle !== null) {
    return true;
  }

  return false;
};

ellipsis配置会影响表头,走的是第一个逻辑

@yusongh
Copy link
Contributor

yusongh commented Sep 18, 2022

如果 #1408 的处理是符合预期的,ellipsis的描述应该需要修改,ellipsis配置并不会控制表头的文字是否省略了。
例如:ellipsisTitle不配置,则为默认值undefined。ellipsis配置为true和false表头文字都是省略的
image
但如果是按照当前ellipsis描述才是符合预期的,应该是判断ellipsisTitle是否配置了,没有配置则取ellipsis的值吧

const isEllipsis = (ellipsis: BaseTableCol['ellipsis'], ellipsisTitle: BaseTableCol['ellipsisTitle']) => {
  if (ellipsisTitle === undefined) {
    return ellipsis;
  }

  if (ellipsisTitle !== false && ellipsisTitle !== null) {
    return true;
  }

  return false;
};

image

如果 #1408 的处理是符合预期的,ellipsis的描述应该需要修改,ellipsis配置并不会控制表头的文字是否省略了。
例如:ellipsisTitle不配置,则为默认值undefined。ellipsis配置为true和false表头文字都是省略的
image
但如果是按照当前ellipsis描述才是符合预期的,应该是判断ellipsisTitle是否配置了,没有配置则取ellipsis的值吧

const isEllipsis = (ellipsis: BaseTableCol['ellipsis'], ellipsisTitle: BaseTableCol['ellipsisTitle']) => {
  if (ellipsisTitle === undefined) {
    return ellipsis;
  }

  if (ellipsisTitle !== false && ellipsisTitle !== null) {
    return true;
  }

  return false;
};

ellipsis配置会影响表头,走的是第一个逻辑

但是目前的现象是,ellipsis设置为false。表头还是会省略哦

image

This was referenced Sep 20, 2022
@chaishi chaishi added need more info need more info and removed 🙏 help wanted Extra attention is needed IssueShoot labels Sep 25, 2022
@Tomaolala
Copy link
Contributor

设置

ellipsis: false,
ellipsisTitle: true,

效果如下图所示,也是符合预期的。不明白标题为什么说 ellipsisTitle 优先级低于 ellipsis,我这边看上去都是正常的

image

当ellipsisTitle默认为undefined 时会省略,ellipsis控制不了

@chaishi
Copy link
Collaborator

chaishi commented Sep 25, 2022

ellipsis设置为false。表头还是会省略哦
需继续修复

@Tomaolala
Copy link
Contributor

已经提了pr了,粗心了

@chaishi
Copy link
Collaborator

chaishi commented Sep 25, 2022

resolved in new change see here

@xiaosansiji
Copy link
Collaborator

已在 0.48.3 版本修复

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
need more info need more info
Projects
None yet
Development

No branches or pull requests

5 participants