Skip to content

feat: activate tabs on hover#273

Open
benzBrake wants to merge 1 commit into
Bush2021:mainfrom
benzBrake:main
Open

feat: activate tabs on hover#273
benzBrake wants to merge 1 commit into
Bush2021:mainfrom
benzBrake:main

Conversation

@benzBrake

Copy link
Copy Markdown

实现 #47 #23 需求

@Bush2021

Bush2021 commented Jul 7, 2026

Copy link
Copy Markdown
Owner

我注意到在你的实现里,每次 WM_MOUSEMOVE 都会走一遍 FindTabHitResult,对于一个热事件来说频繁跨进程 UIA 容易影响性能。

或许可以试试 TrackMouseEvent(TME_HOVER)+ WM_MOUSEHOVER 让 Windows 去判断悬停,这样你也不用自己弄一套 timer。

另外想先说清楚,这个功能会改动原生标签交互并且可能对性能以及日后的维护造成影响,所以我还没拿定该不该加这个功能。你要是有兴趣把它打磨得更好,上面就是我想看到的方向;但最终是否合并我还得再想想。

Comment thread src/uia.cc
need_close_button);
}

bool SelectTab(const TabHitResult& hit_result) {

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

这个思路不错,值得留着。

@benzBrake

benzBrake commented Jul 7, 2026

Copy link
Copy Markdown
Author

我注意到在你的实现里,每次 WM_MOUSEMOVE 都会走一遍 FindTabHitResult,对于一个热事件来说频繁跨进程 UIA 容易影响性能。

或许可以试试 TrackMouseEvent(TME_HOVER)+ WM_MOUSEHOVER 让 Windows 去判断悬停,这样你也不用自己弄一套 timer。

另外想先说清楚,这个功能会改动原生标签交互并且可能对性能以及日后的维护造成影响,所以我还没拿定该不该加这个功能。你要是有兴趣把它打磨得更好,上面就是我想看到的方向;但最终是否合并我还得再想想。

这是 GPT 写的我不懂 C++
甚至测试都要依赖 actions 构建 artifact 后再下载本地测试

@Bush2021

Bush2021 commented Jul 7, 2026

Copy link
Copy Markdown
Owner

8d8478b 怎么 force-push 掉了?

@benzBrake

Copy link
Copy Markdown
Author

8d8478b 怎么 force-push 掉了?

AI 写的整坏了,功能都试下了,我只是利用在线 build

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