Skip to content

fix: Prevent sidebar from interfering with scroll screenshot auto-scroll#770

Merged
deepin-bot[bot] merged 1 commit into
linuxdeepin:develop/snipefrom
dengzhongyuan365-dev:develop/snipe
Dec 2, 2025
Merged

fix: Prevent sidebar from interfering with scroll screenshot auto-scroll#770
deepin-bot[bot] merged 1 commit into
linuxdeepin:develop/snipefrom
dengzhongyuan365-dev:develop/snipe

Conversation

@dengzhongyuan365-dev

Copy link
Copy Markdown
Member
  • Added sidebar area detection in scrollShotMouseMoveEvent() to pause auto-scroll when mouse hovers over sidebar
  • Calculate sidebar rect with pixel ratio when sidebar is visible
  • Updated mouse position check to include sidebar alongside toolbar and shot option areas
  • Added exclusive button group for scroll screenshot toolbar buttons to ensure proper radio button behavior
  • Connected AI Assistant button to scroll button group for

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

- Added sidebar area detection in scrollShotMouseMoveEvent() to pause auto-scroll when mouse hovers over sidebar
- Calculate sidebar rect with pixel ratio when sidebar is visible
- Updated mouse position check to include sidebar alongside toolbar and shot option areas
- Added exclusive button group for scroll screenshot toolbar buttons to ensure proper radio button behavior
- Connected AI Assistant button to scroll button group for

bug: https://pms.uniontech.com/bug-view-342295.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
Member Author

/merge

@deepin-bot

deepin-bot Bot commented Dec 2, 2025

Copy link
Copy Markdown
Contributor

This pr cannot be merged! (status: unstable)

@dengzhongyuan365-dev

Copy link
Copy Markdown
Member Author

/forcemerge

@deepin-bot

deepin-bot Bot commented Dec 2, 2025

Copy link
Copy Markdown
Contributor

This pr force merged! (status: unstable)

@deepin-bot deepin-bot Bot merged commit 7ee8905 into linuxdeepin:develop/snipe Dec 2, 2025
7 of 9 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