feat: support treeland notitlebar mode

linuxdeepin / dtkwidget

Deepin Toolkit, widget module for DDE look and feel

Other

206 stars 144 forks source link

feat: support treeland notitlebar mode #599

Closed justforlxz closed 2 months ago

justforlxz commented 2 months ago

check environment DDE_CURRENT_COMPOSITOR is TreeLand to override DWayland.

Log:

deepin-ci-robot commented 2 months ago

deepin pr auto review

关键摘要:

在DMainWindowPrivate::DMainWindowPrivate函数中，新增的代码块可能会导致titlebar变量未初始化就进行了使用。
DTitlebarPrivate::updateButtonsState函数中，新增的条件判断可能会改变原有的行为，需要确认是否符合预期。
新增代码中使用了qEnvironmentVariable来检查环境变量，但没有考虑跨平台兼容性问题。

是否建议立即修改:

deepin-bot[bot] commented 2 months ago

Doc Check bot :green_circle: Document Coverage Check Passed!

deepin-ci-robot commented 2 months ago

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: justforlxz, kegechen

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

Needs approval from an approver in each of these files: - **[debian/deepin/OWNERS](https://github.com/linuxdeepin/dtkwidget/blob/master/debian/deepin/OWNERS)** Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment