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

feat(TagInput): expand the deletion function of collapsedItems(#2625) #2630

Closed
wants to merge 0 commits into from

Conversation

topazur
Copy link

@topazur topazur commented Nov 30, 2023

🤔 这个 PR 的性质是?

  • 日常 bug 修复
  • 新特性提交
  • 文档改进
  • 演示代码改进
  • 组件样式/交互改进
  • CI/CD 改进
  • 重构
  • 代码风格优化
  • 测试用例
  • 分支合并
  • 其他

🔗 相关 Issue

#2625

💡 需求背景和解决方案

  1. 统一 collapsedItems 的类型定义 (为什么只需要 value 和 onClose?其他属性都可以在外部拿到,且 onClose 仅需要 event 和 index 参数即可)
  2. 统一文档及示例交互形式。
  3. 涉及UI/交互变动需要有截图或 GIF。
image

📝 更新日志

  • feat(TagInput): expand the deletion function of collapsedItems

  • 本条 PR 不需要纳入 Changelog

☑️ 请求合并前的自查清单

⚠️ 请自检并全部勾选全部选项⚠️

  • 文档已补充或无须补充
  • 代码演示已提供或无须提供
  • TypeScript 定义已补充或无须补充
  • Changelog 已提供或无须提供

Copy link
Contributor

github-actions bot commented Dec 2, 2023

失败

@topazur
Copy link
Author

topazur commented Dec 8, 2023

@uyarn 你好,已提交函数collapsedItems的参数兼容版本,通过后再去修改vue的pr。

注:目前 collapsedSelectedItems 和 collapsedTags 混淆使用。参考 vue 是以 collapsedSelectedItems 参数,因此修改了文档及示例名词,但是在参数传递时同时传递了 collapsedSelectedItems 和 collapsedTags 两个参数,应该可以达到兼容的目的。

@uyarn
Copy link
Collaborator

uyarn commented Dec 14, 2023

@uyarn 你好,已提交函数collapsedItems的参数兼容版本,通过后再去修改vue的pr。

注:目前 collapsedSelectedItems 和 collapsedTags 混淆使用。参考 vue 是以 collapsedSelectedItems 参数,因此修改了文档及示例名词,但是在参数传递时同时传递了 collapsedSelectedItems 和 collapsedTags 两个参数,应该可以达到兼容的目的。

好的

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

Successfully merging this pull request may close these issues.

2 participants