-
Notifications
You must be signed in to change notification settings - Fork 178
add linux support for ShowSystemMenu #203
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
wangwenx190
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
代码没有大问题,但有些小细节可能需要注意下
感谢,能支持最新版本就挺好了,我们会在稍微老一点的版本上测试一下。最主要的问题应该是此版代码不支持qt5,不过qwk在Linux上是否需要支持qt5本身也是有待商榷。 |
wangwenx190
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
辛苦了,感谢!还是有些细节有待商榷
|
麻烦老哥了,我又重新修改整成一个提交了,有时间再审查一下。那个协议版权声明顺序错的范围有点太大了,我不想改了,我只改动了我 PR 涉及的部分。 |
wangwenx190
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
真是辛苦你了,只有一处小问题了
没问题的,辛苦了,其他没有涉及的文件确实不必改,否则这个PR的内容就有些不明确了 |
|
辛苦了,我认为没什么问题了。我让另一位维护者也看一下 @SineStriker |
wangwenx190
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. +2
|
直接引 x11 和 wayland 的头文件应该就不需要定义这些结构了吧,在 xlibint 里有。 |
|
linuxdesktopenvapi.h 最好不要有这个文件,能不能写到 qwindowkit_linux.h 里去,目前这个文件还是空的。 |
xlib 在以后 Wayland 为唯一桌面环境的时候,可能不会存在。如果直接引用头文件,这就要求用户需要安装对应的包。里面用的结构体也不算太多,而且 x11 已经可以说稳定了,应该不会有结构体上的改动。 对于 Wayland 来说, qt 提供的前置声明就够用了,相关函数也只是用到了一些结构体指针和宏定义的枚举。 |
这个没问题,今晚我可以仿照 Windows 写的方式写一下。 |
你好,linuxdesktopenvapi 相关的内容已移动到 qwindowkit_linux 相关的内容,麻烦再看看有没有其他问题。 |
|
我认为没什么问题 |
相关讨论为 #202 ,本 PR 提交实现的功能和修复说明:
注意事项:
_GTK_SHOW_WINDOW_MENU,如果不支持则无效xdg_toplevel以及XDG_TOPLEVEL_SHOW_WINDOW_MENU,如果不支持则无效测试:
该补丁在 Qt6.10.1 + KDE 测试通过,其他 Qt 版本和 Linux 桌面可能需要仓库拥有者以及主要维护者的测试。