Skip to content

Add search support for process list in binding page#116

Open
Kill1ngPig wants to merge 3 commits into
IvanHanloth:devfrom
Kill1ngPig:feat/process-search
Open

Add search support for process list in binding page#116
Kill1ngPig wants to merge 3 commits into
IvanHanloth:devfrom
Kill1ngPig:feat/process-search

Conversation

@Kill1ngPig
Copy link
Copy Markdown

为窗口绑定页面左侧“现有窗口进程”列表增加搜索功能,支持按进程名或窗口标题进行动态过滤,清空搜索后可恢复完整列表。

为保证搜索与原有添加/删除绑定、刷新进程等逻辑的一致性,引入了左侧完整窗口数据缓存 left_all_windows。搜索时基于缓存数据进行过滤,并统一通过 InsertTreeList() 重新渲染列表,而不直接操作树控件。

这样可以确保搜索始终基于完整数据,同时避免在多种操作场景下出现数据不一致的问题。部分实现了issue #113 。本次修改仅涉及搜索功能,不包含默认展开/收起行为的调整。

@IvanHanloth IvanHanloth requested a review from Copilot April 15, 2026 13:28
Copy link
Copy Markdown

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

Adds search/filtering to the left-side “现有窗口进程” list on the binding page by introducing a cached full dataset and re-rendering the tree based on the current search keyword.

Changes:

  • Added a wx.SearchCtrl above the left process/window list and wired it to filter dynamically.
  • Introduced left_all_windows cache and centralized left-list rendering through ApplyLeftFilter() + InsertTreeList().
  • Updated add/remove binding flows to keep the left cache in sync and re-apply the current filter.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread main/GUI/setting/binding_page.py
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.

3 participants