Skip to content
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.

Sign up for GitHub

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

Jump to bottom

refactor: port to Qt6, new Wayland implementation #176

Merged
merged 1 commit into from
Nov 18, 2024
Merged

refactor: port to Qt6, new Wayland implementation #176

merged 1 commit into from
Nov 18, 2024

Conversation

RigoLigoRLC
Copy link
Contributor

@RigoLigoRLC RigoLigoRLC commented Nov 5, 2024 •
edited
Loading

  • 将项目整体移植到 Qt6 进行编译
  • 弃用了旧的 CMake D-Bus 代码模板生成方式，使用 Qt6 提供的新 API 实现
  • 移除了旧的 Wayland 依赖库及其后端，重新使用 wlr_data_control API 为其实现了 Wayland 后端
  • 使用 dde-shell 的 layer-shell API 为前端实现窗口贴靠
  • 修复部分因为缺少运算符重载在本地产生的编译失败
  • 修复了部分 Qt6 下缺失的过时 API 调用

遗留问题：移植到 layer-shell 之后，无法使用 setFixedSize 固定窗口大小，为了使窗口不可变尺寸修改了部分 window flags，导致 compositor 不再为窗口显示隐藏使用动画。

@deepin-ci-robot
Copy link

Hi @RigoLigoRLC. Thanks for your PR. 😃

@deepin-ci-robot
Copy link

Hi @RigoLigoRLC. Thanks for your PR.

I'm waiting for a linuxdeepin member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@RigoLigoRLC RigoLigoRLC force-pushed the port-qt6-master branch 2 times, most recently from 0d76945 to 07c5ce2 Compare November 5, 2024 07:35
BLumia
BLumia previously approved these changes Nov 5, 2024
View reviewed changes
@BLumia BLumia self-requested a review November 5, 2024 07:59
@BLumia BLumia dismissed their stale review November 5, 2024 07:59

现有问题需要改完

@RigoLigoRLC RigoLigoRLC force-pushed the port-qt6-master branch 4 times, most recently from 8218be2 to 007ff1d Compare November 5, 2024 08:17
BLumia
BLumia previously approved these changes Nov 5, 2024
View reviewed changes
@tsic404
Copy link
Contributor

tsic404 commented Nov 6, 2024

/test all

@deepin-ci-robot
Copy link

deepin pr auto review

--
2.25.1

@RigoLigoRLC RigoLigoRLC force-pushed the port-qt6-master branch 4 times, most recently from 2cdc780 to 4cd5934 Compare November 6, 2024 06:51
@github-actions GitHub Actions
Copy link

github-actions bot commented Nov 6, 2024

TAG Bot

TAG: 6.1.0
EXISTED: no
DISTRIBUTION: unstable

@RigoLigoRLC
Copy link
Contributor Author

/integrate

@github-actions GitHub Actions
Copy link

github-actions bot commented Nov 6, 2024

AutoIntegrationPr Bot
auto integrate with pr url: deepin-community/Repository-Integration#2186
PrNumber: 2186
PrBranch: auto-integration-11698839349

@deepin-community-ci-bot deepin-community-ci-bot bot mentioned this pull request Nov 6, 2024
Open
@RigoLigoRLC RigoLigoRLC force-pushed the port-qt6-master branch 4 times, most recently from e52d049 to 8d74f4d Compare November 8, 2024 01:49
@RigoLigoRLC
Copy link
Contributor Author

/integrate

@RigoLigoRLC
refactor: port to Qt6, new Wayland implementation
8d146b8 
- 将项目整体移植到 Qt6 进行编译
- 弃用了旧的 CMake D-Bus 代码模板生成方式，使用 Qt6 提供的新 API 实现
- 移除了旧的 Wayland 依赖库及其后端，重新使用 wlr_data_device API 为其实现了 Wayland 后端
- 使用 dde-shell 的 layer-shell API 为前端实现窗口贴靠
- 修复部分因为缺少运算符重载在本地产生的编译失败
- 修复了部分 Qt6 下缺失的过时 API 调用

Log:
@RigoLigoRLC
Copy link
Contributor Author

/integrate

@deepin-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: RigoLigoRLC, yixinshark

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@yixinshark yixinshark merged commit 05abc19 into linuxdeepin:master Nov 18, 2024
17 checks passed
This was referenced Nov 18, 2024
Closed
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@yixinshark yixinshark yixinshark approved these changes

@BLumia BLumia BLumia left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@RigoLigoRLC @deepin-ci-robot @tsic404 @BLumia @yixinshark