Skip to content

增加 KDE Plasma 6.8+ 支持,修复贴图拖动 - #9

Merged
kregonia merged 4 commits into
kregonia:mainfrom
lhs-12:feat/kde-plasma
Oct 20, 2026
Merged

kregonia merged 4 commits into
kregonia:mainfrom
lhs-12:feat/kde-plasma

Conversation

@lhs-12

@lhs-12 lhs-12 commented Oct 20, 2026

Copy link
Copy Markdown
Contributor

这个 PR 包含两个 commit:

  • 增加 KDE Plasma 6.8+ 支持,拆分桌面逻辑为策略接口。
  • 修复贴图拖动抖动和不跟手。

KDE 后端目前只支持 Plasma 6.8+。

已在 KDE 6.8 测试,需要 niri/Hyprland 用户帮忙回归测试。

lhs-12 and others added 4 commits October 20, 2026 14:42
区分隐式抓取和表面局部坐标,避免重复 motion 引发位移正反馈;在松手前应用同帧末次移动,几何不变时跳过重复提交。新增连续事件、四边四角、反向与边界回归测试,并在 niri 和 KWin/Plasma 6.8 的独立 Wayland 会话验证。
@kregonia
kregonia merged commit e252e69 into kregonia:main Oct 20, 2026
@lhs-12

lhs-12 commented Oct 20, 2026 •

Copy link
Copy Markdown
Contributor Author

@kregonia 我在 KDE 上又回归了一遍最新 commit, 暂时没发现问题.

补充说明: 写代码过程中 AI 审查提到过一些点, 但其中一部分我没有对应环境验证, 所以没改. 这里列个大概, 细节你可以根据自己需要让 AI 调查:

  1. niri/Hyprland 的真机回归不足且缺乏单元测试 (这个你已经回归测试测试并修复 niri)
  2. niri 的输出解析单测依赖 JSON map 的遍历顺序, 可能需要改成按输出名查找
  3. KDE 多显示器没有验证, 尤其是指定非焦点输出时, 编辑窗口的全屏落点可能仍在焦点屏
  4. 截图全屏目前只在 KDE 调用(set_fullscreen(true)), niri/Hyprland 保留各自原行为, 避免影响我无法验证的环境, 是否统一成公共行为由你判断
  5. 按住拖动时同时滚轮缩放的话可能会冲突. 但一般用户不会这么操作, 暂时不处理.

这些都不是这个 PR 的阻塞项, 列出来供后续参考. 也感谢你的开发修复~

顺带一提我的一些想法:

  1. 目前命令调用的默认行为会保存文件到 ~/Pictures , 我觉得默认无参数行为可能不落盘更好, 毕竟已经有ctrl+s快捷键或保存文件参数调用的方式去保存了
  2. 贴图的拖动一定限制在屏幕内感觉有时候可能没那么友好? 比如同时有多个贴图, 或者贴图面积大, 桌面空间不够, 有时候需要将贴图一部分暂时超过桌面范围, 但现在只能缩小图片在屏幕内移动

@lhs-12
lhs-12 deleted the feat/kde-plasma branch October 20, 2026 17:47
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