weasel icon indicating copy to clipboard operation
weasel copied to clipboard

fix(WeaselUI): fix #1580, tip is shown incorrectly[wip]

Open fxliang opened this issue 7 months ago • 5 comments

fix #1580

fxliang avatar May 22 '25 10:05 fxliang

邏輯好複雜,看不懂了。你多測測。

lotem avatar May 22 '25 10:05 lotem

邏輯好複雜,看不懂了。你多測測。

我也觉得,好久之前写的可以漏测了隐藏窗口这个状态。先挂上,观察下没太大问题就合了后面脑子清晰一点再看怎么理一下。

fxliang avatar May 22 '25 10:05 fxliang

还有点问题

fxliang avatar May 23 '25 01:05 fxliang

我不明白。 爲什麼我最初寫好的代碼不能一直用到鏽。 不過只要能修好,我也沒意見。

lotem avatar May 23 '25 04:05 lotem

我不明白。 爲什麼我最初寫好的代碼不能一直用到鏽。 不過只要能修好,我也沒意見。

这个大概是这样的,原来当margin_x或margin_y设置为负数的时候会绘制一些奇怪的状态,在设定成一个绝对值足够小的负值时可以靠bug实现隐藏候选窗口的功能

然后我后来重构ui的时候将这个bug产生的特性再正经化一点,只要margin_x或者margin_y负数就隐藏候选,但是要在方案选单或者tip的时候不隐藏。原来的代码少留意了这个隐藏窗口的时候的tip显示(只显示icon或者option切换信息),于是终于在这次发版的时候有用户提出来了。

目前这个修的,可能还是不够好,观察一段时间我看有没有更好的修理办法了。

fxliang avatar May 23 '25 05:05 fxliang