Sitelet https://github.com/flutter/flutter/pull/190029
Skip to content

[Win32] Ignore mouse move events without capture when dragging - #190029

Merged
knopp merged 2 commits into
flutter:masterfrom
knopp:fix_mouse
Jul 29, 2026
Merged

knopp merged 2 commits into
flutter:masterfrom
knopp:fix_mouse

Conversation

@knopp

@knopp knopp commented Jul 25, 2026

Copy link
Copy Markdown
Member

Fixes #189583

Pre-launch Checklist

If you need help, consider asking for advice on the #hackers-new channel on Discord.

If this change needs to override an active code freeze, provide a comment explaining why. The code freeze workflow can be overridden by code reviewers. See pinned issues for any active code freezes with guidance.

Note: The Flutter team is currently trialing the use of Gemini Code Assist for GitHub. Comments from the gemini-code-assist bot should not be taken as authoritative feedback from the Flutter team. If you find its comments useful you can update your code accordingly, but if you are unsure or disagree with the feedback, please feel free to wait for a Flutter team member's review for guidance on which automated comments should be addressed.

@knopp
knopp requested a review from a team as a code owner July 25, 2026 18:56
@flutter-dashboard flutter-dashboard Bot added the CICD Run CI/CD label Jul 25, 2026
@github-actions github-actions Bot added engine flutter/engine related. See also e: labels. platform-windows Building on or for Windows specifically a: desktop Running on desktop team-windows Owned by the Windows platform team labels Jul 25, 2026
@knopp knopp changed the title [Win32] Ignore mouse move events without capture when draggin [Win32] Ignore mouse move events without capture when dragging Jul 25, 2026
@knopp
knopp requested review from loic-sharma and mattkae July 25, 2026 18:57

@gemini-code-assist gemini-code-assist Bot left a comment

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.

Code Review

This pull request updates the Windows platform code to ignore mouse move events when buttons are pressed but the window does not have capture, addressing issues when switching windows mid-drag. Additionally, unit tests are added to verify this behavior under both captured and non-captured states. There are no review comments, and I have no feedback to provide.

Comment thread engine/src/flutter/shell/platform/windows/flutter_window.cc

@loic-sharma loic-sharma left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks for the fix!

auto-submit Bot pushed a commit to flutter/packages that referenced this pull request Aug 8, 2026
…12393)

Manual roll requested by stuartmorgan@google.com

flutter/flutter@2a230d1...2757a77

2026-07-29 engine-flutter-autoroll@skia.org Roll Skia from 3ae9e364d30b to 0f35bba4945c (2 revisions) (flutter/flutter#190215)
2026-07-29 kustermann@google.com [web] Fix deferred loading with wasm (flutter/flutter#190140)
2026-07-29 engine-flutter-autoroll@skia.org Roll Dart SDK from 1bbaacba6a5a to 43f7de5f4977 (1 revision) (flutter/flutter#190211)
2026-07-29 engine-flutter-autoroll@skia.org Roll Skia from 119cae396625 to 3ae9e364d30b (6 revisions) (flutter/flutter#190208)
2026-07-29 matej.knopp@gmail.com [Win32] Ignore mouse move events without capture when dragging (flutter/flutter#190029)
2026-07-29 engine-flutter-autoroll@skia.org Roll Dart SDK from 49c637261348 to 1bbaacba6a5a (7 revisions) (flutter/flutter#190196)
2026-07-29 engine-flutter-autoroll@skia.org Roll Skia from 30ad01017a46 to 119cae396625 (2 revisions) (flutter/flutter#190183)
2026-07-29 46920873+gabrimatic@users.noreply.github.com Fix NestedScrollView example crash when switching tabs on desktop (flutter/flutter#186993)
2026-07-29 engine-flutter-autoroll@skia.org Roll Fuchsia Linux SDK from OZkZC_2CZ_G5rbMIS... to J8rVTlBnjnUpuHYk3... (flutter/flutter#190174)
2026-07-29 git@schultek.dev Update widget location tracking to use the new generalized api (flutter/flutter#189725)
2026-07-29 bernaferrari2@gmail.com Avoid unnecessary work in `BoxDecoration` and `RenderPhysicalModel` (flutter/flutter#187891)
2026-07-29 me@davidmiguel.com Fix web context-menu client lifecycle in SelectableRegion (flutter/flutter#189587)
2026-07-29 engine-flutter-autoroll@skia.org Roll Skia from 70733f74d415 to 30ad01017a46 (2 revisions) (flutter/flutter#190173)
2026-07-29 jason-simmons@users.noreply.github.com Manual roll of Dart from 28e63ac22d8d to 49c637261348 (flutter/flutter#190158)

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-packages
Please CC boetger@google.com,stuartmorgan@google.com on the revert to ensure that a human
is aware of the problem.

To file a bug in Packages: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
palmoni5 added a commit to palmoni5/otzaria that referenced this pull request Sep 20, 2026
התיקון ב-engine (flutter/flutter#190029) נכלל ב-3.48 ומייתר את המסנן שנוסף ב-issue Otzaria#1441.
Y-PLONI pushed a commit to palmoni5/otzaria that referenced this pull request Sep 22, 2026
התיקון ב-engine (flutter/flutter#190029) נכלל ב-3.48 ומייתר את המסנן שנוסף ב-issue Otzaria#1441.
@mattkae mattkae added the cp: stable cherry pick this pull request to stable release candidate branch label Sep 23, 2026
@flutteractionsbot

Copy link
Copy Markdown
Contributor

Failed to create CP due to merge conflicts.
You will need to create the PR manually. See the cherrypick wiki for more info.

@alexskim alexskim mentioned this pull request Oct 4, 2026
5 of 6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

a: desktop Running on desktop CICD Run CI/CD cp: stable cherry pick this pull request to stable release candidate branch engine flutter/engine related. See also e: labels. platform-windows Building on or for Windows specifically team-windows Owned by the Windows platform team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Windows] Multi-window: buttons stop working when dragging a click from a window to another one

4 participants