Skip to content

fix(date-picker): popupProps.onVisibleChange not fired on popup close - #6774

Open
RSS1102 wants to merge 13 commits into
developfrom
rss1102/fix/date-range-picker/cell-click/on-visible-change
Open

fix(date-picker): popupProps.onVisibleChange not fired on popup close#6774
RSS1102 wants to merge 13 commits into
developfrom
rss1102/fix/date-range-picker/cell-click/on-visible-change

Conversation

@RSS1102

@RSS1102 RSS1102 commented Jul 2, 2026

Copy link
Copy Markdown
Collaborator

🤔 这个 PR 的性质是?

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

🔗 相关 Issue

TDesignOteam/tdesign-api#920

💡 需求背景和解决方案

📝 更新日志

  • 本条 PR 不需要纳入 Changelog

tdesign-vue-next

  • fix(date-picker): 修复关闭面板时未正确触发 popupProps.onVisibleChange 的问题,关闭场景统一使用 trigger: 'trigger-element-close',并在用户交互关闭时透传原始事件 e
  • fix(popup): 修正 trigger 元素隐藏时关闭事件的 trigger 来源,避免将内部关闭错误标记为 trigger-element-hover

@tdesign-vue-next/chat

@tdesign-vue-next/nuxt

@tdesign-vue-next/auto-import-resolver

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

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

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

@github-actions

github-actions Bot commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

Size Change: 0 B

Total Size: 21.4 kB

ℹ️ View Unchanged
Filename Size
packages/auto-import-resolver/dist/index.cjs 10.7 kB
packages/auto-import-resolver/dist/index.js 10.7 kB

compressed-size-action::auto-import-resolver

@github-actions

github-actions Bot commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

Size Change: 0 B

Total Size: 3.55 kB

ℹ️ View Unchanged
Filename Size
packages/tdesign-vue-next-nuxt/dist/module.d.mts 551 B
packages/tdesign-vue-next-nuxt/dist/module.mjs 2.84 kB
packages/tdesign-vue-next-nuxt/dist/types.d.mts 164 B

compressed-size-action::tdesign-vue-next-nuxt

@github-actions

github-actions Bot commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

Size Change: 0 B

Total Size: 74.6 kB

ℹ️ View Unchanged
Filename Size
packages/tdesign-vue-next-chat/es/_chunks/dep-aa0ac517.mjs 30.4 kB
packages/tdesign-vue-next-chat/es/_chunks/dep-ac45875e.mjs 2.16 kB
packages/tdesign-vue-next-chat/es/attachments/index.mjs 240 B
packages/tdesign-vue-next-chat/es/chat-actionbar/chat-actionbar-props.mjs 364 B
packages/tdesign-vue-next-chat/es/chat-actionbar/chat-actionbar.mjs 1.57 kB
packages/tdesign-vue-next-chat/es/chat-actionbar/index.mjs 335 B
packages/tdesign-vue-next-chat/es/chat-content/chat-content-props.mjs 358 B
packages/tdesign-vue-next-chat/es/chat-content/chat-content.mjs 2.23 kB
packages/tdesign-vue-next-chat/es/chat-content/index.mjs 384 B
packages/tdesign-vue-next-chat/es/chat-engine/components/activity/index.mjs 321 B
packages/tdesign-vue-next-chat/es/chat-engine/components/activity/registry.mjs 406 B
packages/tdesign-vue-next-chat/es/chat-engine/components/activity/render.mjs 784 B
packages/tdesign-vue-next-chat/es/chat-engine/components/provider/agent-state.mjs 521 B
packages/tdesign-vue-next-chat/es/chat-engine/components/provider/index.mjs 219 B
packages/tdesign-vue-next-chat/es/chat-engine/components/shared/ComponentErrorBoundary.mjs 549 B
packages/tdesign-vue-next-chat/es/chat-engine/components/shared/createRegistry.mjs 978 B
packages/tdesign-vue-next-chat/es/chat-engine/components/shared/index.mjs 258 B
packages/tdesign-vue-next-chat/es/chat-engine/components/shared/useRegistrationListener.mjs 624 B
packages/tdesign-vue-next-chat/es/chat-engine/components/toolcall/hoc.mjs 755 B
packages/tdesign-vue-next-chat/es/chat-engine/components/toolcall/index.mjs 441 B
packages/tdesign-vue-next-chat/es/chat-engine/components/toolcall/registry.mjs 410 B
packages/tdesign-vue-next-chat/es/chat-engine/components/toolcall/render.mjs 339 B
packages/tdesign-vue-next-chat/es/chat-engine/hooks/index.mjs 345 B
packages/tdesign-vue-next-chat/es/chat-engine/hooks/useAgentActivity.mjs 953 B
packages/tdesign-vue-next-chat/es/chat-engine/hooks/useAgentState.mjs 1.22 kB
packages/tdesign-vue-next-chat/es/chat-engine/hooks/useAgentToolcall.mjs 920 B
packages/tdesign-vue-next-chat/es/chat-engine/hooks/useChat.mjs 686 B
packages/tdesign-vue-next-chat/es/chat-engine/index.mjs 560 B
packages/tdesign-vue-next-chat/es/chat-input/chat-input-props.mjs 352 B
packages/tdesign-vue-next-chat/es/chat-input/chat-input.mjs 1.53 kB
packages/tdesign-vue-next-chat/es/chat-input/index.mjs 321 B
packages/tdesign-vue-next-chat/es/chat-item/chat-item-props.mjs 431 B
packages/tdesign-vue-next-chat/es/chat-item/chat-item.mjs 1.98 kB
packages/tdesign-vue-next-chat/es/chat-item/index.mjs 432 B
packages/tdesign-vue-next-chat/es/chat-list/chat-list.mjs 2.78 kB
packages/tdesign-vue-next-chat/es/chat-list/index.mjs 365 B
packages/tdesign-vue-next-chat/es/chat-list/props.mjs 454 B
packages/tdesign-vue-next-chat/es/chat-loading/chat-loading-props.mjs 281 B
packages/tdesign-vue-next-chat/es/chat-loading/index.mjs 244 B
packages/tdesign-vue-next-chat/es/chat-markdown/index.mjs 260 B
packages/tdesign-vue-next-chat/es/chat-message/chat-message-props.mjs 436 B
packages/tdesign-vue-next-chat/es/chat-message/chat-message.mjs 1.25 kB
packages/tdesign-vue-next-chat/es/chat-message/index.mjs 318 B
packages/tdesign-vue-next-chat/es/chat-reasoning/chat-reasoning-props.mjs 421 B
packages/tdesign-vue-next-chat/es/chat-reasoning/chat-reasoning.mjs 1.19 kB
packages/tdesign-vue-next-chat/es/chat-reasoning/index.mjs 300 B
packages/tdesign-vue-next-chat/es/chat-sender/chat-sender-props.mjs 366 B
packages/tdesign-vue-next-chat/es/chat-sender/chat-sender.mjs 3.02 kB
packages/tdesign-vue-next-chat/es/chat-sender/index.mjs 357 B
packages/tdesign-vue-next-chat/es/chat-thinking/chat-thinking-props.mjs 387 B
packages/tdesign-vue-next-chat/es/chat-thinking/chat-thinking.mjs 942 B
packages/tdesign-vue-next-chat/es/chat-thinking/index.mjs 334 B
packages/tdesign-vue-next-chat/es/chatbot/index.mjs 397 B
packages/tdesign-vue-next-chat/es/config-provider.mjs 158 B
packages/tdesign-vue-next-chat/es/index.mjs 1.51 kB
packages/tdesign-vue-next-chat/es/style/css.mjs 42 B
packages/tdesign-vue-next-chat/es/style/index.css 3.97 kB
packages/tdesign-vue-next-chat/es/utils/index.mjs 494 B

compressed-size-action::tdesign-vue-next-chat

@tdesign-bot

tdesign-bot commented Jul 2, 2026

Copy link
Copy Markdown
Collaborator

TDesign Component Site Preview Open

Component Preview
tdesign-vue-next 完成
@tdesign-vue-next/chat 完成

@pkg-pr-new

pkg-pr-new Bot commented Jul 2, 2026

Copy link
Copy Markdown
  • tdesign-vue-next-demo

    npm i https://pkg.pr.new/@tdesign-vue-next/auto-import-resolver@6774
    
    npm i https://pkg.pr.new/tdesign-vue-next@6774
    
    npm i https://pkg.pr.new/@tdesign-vue-next/chat@6774
    
    npm i https://pkg.pr.new/@tdesign-vue-next/nuxt@6774
    

commit: dceeae9

@github-actions

github-actions Bot commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

Size Change: +885 B (+0.09%)

Total Size: 954 kB

📦 View Changed
Filename Size Change
packages/tdesign-vue-next/dist/tdesign.js 529 kB +543 B (+0.1%)
packages/tdesign-vue-next/dist/tdesign.min.js 315 kB +342 B (+0.11%)
ℹ️ View Unchanged
Filename Size
packages/tdesign-vue-next/dist/plugin.css 3.02 kB
packages/tdesign-vue-next/dist/reset.css 1.91 kB
packages/tdesign-vue-next/dist/tdesign.css 55.4 kB
packages/tdesign-vue-next/dist/tdesign.min.css 50.3 kB

compressed-size-action::tdesign-vue-next

@RSS1102
RSS1102 marked this pull request as draft July 2, 2026 09:44
@RSS1102 RSS1102 changed the title fix(date-range-picker): 选完日期关闭面板时触发 onVisibleChange fix(date-range-picker): trigger onVisibleChange when panel closes after date selection Jul 5, 2026
@RSS1102 RSS1102 changed the title fix(date-range-picker): trigger onVisibleChange when panel closes after date selection fix(date-picker): trigger onVisibleChange when panel closes after date selection Jul 5, 2026
@RSS1102
RSS1102 marked this pull request as ready for review July 7, 2026 08:02
@RSS1102 RSS1102 changed the title fix(date-picker): trigger onVisibleChange when panel closes after date selection fix(date-picker): popupProps.onVisibleChange not fired on popup close Jul 8, 2026
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