Skip to content

Conversation

@RylanBot
Copy link
Collaborator

@RylanBot RylanBot commented Sep 9, 2025

🤔 这个 PR 的性质是?

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

🔗 相关 Issue

#3769

💡 需求背景和解决方案

📝 更新日志

  • fix(Table): 修复过滤后的数据量在跨越虚拟滚动的 threshold 时,筛选器下拉框被强行关闭的问题

  • 本条 PR 不需要纳入 Changelog

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

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

  • 文档已补充或无须补充
  • 代码演示已提供或无须提供
  • TypeScript 定义已补充或无须补充
  • Changelog 已提供或无须提供
@pkg-pr-new
Copy link

pkg-pr-new bot commented Sep 9, 2025

tdesign-react-demo

npm i https://pkg.pr.new/tdesign-react@3803 

commit: 6c3f10a

@github-actions
Copy link
Contributor

github-actions bot commented Sep 9, 2025

完成

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR fixes an issue where filter dropdown menus in Table components are forcibly closed when filtered data crosses the virtual scrolling threshold. The fix involves unifying popup visibility handling across filter components by centralizing state management and improving the visibility change detection logic.

  • Centralized popup visibility state management in the useFilter hook
  • Enhanced popup visibility change detection to prevent unwanted closures
  • Modified FilterController to accept external visibility state and colKey context

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
packages/components/table/hooks/useFilter.tsx Added centralized popup visibility state management and updated visibility change handler
packages/components/table/FilterController.tsx Modified to accept external visibility state and improved popup close detection logic

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants