Skip to content

feat: support move xembed window on treeland#1573

Draft
BLumia wants to merge 1 commit intolinuxdeepin:masterfrom
BLumia:xembed-treeland-move
Draft

feat: support move xembed window on treeland#1573
BLumia wants to merge 1 commit intolinuxdeepin:masterfrom
BLumia:xembed-treeland-move

Conversation

@BLumia
Copy link
Copy Markdown
Member

@BLumia BLumia commented Apr 28, 2026

通过 treeland 提供的 API 移动 xembed 窗口。

@BLumia BLumia requested a review from tsic404 April 28, 2026 09:02
@deepin-ci-robot
Copy link
Copy Markdown

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

Copy link
Copy Markdown

@sourcery-ai sourcery-ai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry @BLumia, you have reached your weekly rate limit of 500000 diff characters.

Please try again later or upgrade to continue using Sourcery

@deepin-ci-robot
Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: BLumia

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

Details 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

通过 treeland 提供的 API 移动 xembed 窗口

Log:
@BLumia BLumia force-pushed the xembed-treeland-move branch from 18de6a1 to d0bfc61 Compare April 28, 2026 09:44
PluginSurface *pluginSurface = findPluginSurface(plugin_id, item_key);
if (pluginSurface) {
dx = pluginSurface->itemPosition().x();
dy = pluginSurface->itemPosition().y();
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这个 itemPosition 是在任务栏上的位置偏移吗?如果托盘是在展开区域的话,这个位置正确吗?

return false;
}

m_ddeShellManager->setXWindowPositionRelative(wid, m_dockWlSurface, dx, dy);
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

此处都是基于 dockWlSurface 进行偏移,是否有考虑托盘在展开区域情况呢?

@deepin-bot
Copy link
Copy Markdown

deepin-bot Bot commented Apr 29, 2026

TAG Bot

New tag: 2.0.39
DISTRIBUTION: unstable
Suggest: synchronizing this PR through rebase #1580

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