Skip to content

fix(shot): include sidebar in background clip region for blur effect#816

Merged
deepin-bot[bot] merged 1 commit intolinuxdeepin:masterfrom
dengzhongyuan365-dev:master
Apr 11, 2026
Merged

fix(shot): include sidebar in background clip region for blur effect#816
deepin-bot[bot] merged 1 commit intolinuxdeepin:masterfrom
dengzhongyuan365-dev:master

Conversation

@dengzhongyuan365-dev
Copy link
Copy Markdown
Contributor

In recording/scrollshot modes, the background is only drawn within the toolbar clip area to avoid blocking mouse passthrough. When the AI assistant panel (sidebar) is shown, it falls outside the toolbar clip rect, so its blur backdrop has no background pixels to sample from.

Extend the clip region to also cover the sidebar geometry when it is visible, using QRegion to combine both rects.

bug: https://pms.uniontech.com/bug-view-356057.html

In recording/scrollshot modes, the background is only drawn within
the toolbar clip area to avoid blocking mouse passthrough. When the
AI assistant panel (sidebar) is shown, it falls outside the toolbar
clip rect, so its blur backdrop has no background pixels to sample
from.

Extend the clip region to also cover the sidebar geometry when it is
visible, using QRegion to combine both rects.

bug: https://pms.uniontech.com/bug-view-356057.html
@deepin-ci-robot
Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: dengzhongyuan365-dev, lzwind

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

@dengzhongyuan365-dev
Copy link
Copy Markdown
Contributor Author

/merge

@deepin-bot
Copy link
Copy Markdown
Contributor

deepin-bot bot commented Apr 11, 2026

This pr cannot be merged! (status: unstable)

@dengzhongyuan365-dev
Copy link
Copy Markdown
Contributor Author

/forcemerge

@deepin-bot
Copy link
Copy Markdown
Contributor

deepin-bot bot commented Apr 11, 2026

This pr force merged! (status: unstable)

@deepin-bot deepin-bot bot merged commit b8054be into linuxdeepin:master Apr 11, 2026
8 of 10 checks passed
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