Add tooltip windows to the windowing API alongside the window positioning logic by mattkae · Pull Request #177404 · flutter/flutter

mattkae · 2025-10-22T19:04:30Z

What's new?

Added tooltips to the windowing API, but they have no implementation for the time being
Added the WindowPostiioner logic to the windowing API
Wrote a lot of tests 🧪

Pre-launch Checklist

I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
I read the [Tree Hygiene] wiki page, which explains my responsibilities.
I read and followed the [Flutter Style Guide], including [Features we expect every widget to implement].
I signed the [CLA].
I listed at least one issue that this PR fixes in the description above.
I updated/added relevant documentation (doc comments with ///).
I added new tests to check the change I am making, or this PR is [test-exempt].
I followed the [breaking change policy] and added [Data Driven Fixes] where supported.
All existing and new tests are passing.

…ning logic

gemini-code-assist

Code Review

This pull request introduces tooltip windows to the windowing API, along with the logic for positioning them. A significant amount of new code has been added, including the TooltipWindowController, TooltipWindow widget, and the WindowPositioner logic. A comprehensive suite of tests for the new window positioning logic has also been included.

My review focuses on improving code clarity, consistency, and maintainability. I've pointed out a few documentation typos and copy-paste errors, including a misleading error message in the Win32 implementation. I've also suggested a minor refactoring in the WindowPositioner to improve code readability and reduce redundancy. Overall, this is a solid addition with good test coverage.

packages/flutter/lib/src/widgets/_window_win32.dart

packages/flutter/lib/src/widgets/_window.dart

packages/flutter/lib/src/widgets/_window_positioner.dart

packages/flutter/lib/src/widgets/_window.dart

packages/flutter/lib/src/widgets/_window_positioner.dart

packages/flutter/lib/src/widgets/_window.dart

packages/flutter/lib/src/widgets/_window_positioner.dart

…f an enum

justinmc

Overall looks good. This kind of positioning stuff is fiddly but I think your way of doing things keeps it pretty clean.

My main concern is with the use of GlobalKeys in the example and how that should be done in the ideal situation, because I want to make sure we set users up to be able to get this right on the happy path.

I see that the framework is positioning the tooltip based on constraints from the platform. Just to make sure, do all platforms provide this kind of constraint size information?

packages/flutter/lib/src/widgets/_window.dart

packages/flutter/lib/src/widgets/_window_positioner.dart

packages/flutter/test/widgets/window_positioner_test.dart

mattkae · 2025-11-19T21:39:28Z

Alignment

@justinmc After talking with Saviq about this, we've settled that having anchor enums (both child and parent) are important enough to justify the new constructs. When we apply the constraints for off-window tooltips/popups, it will make more sense to do so with predefined anchors. A general Alignment might lead us to do something inconsistent.

For the "teardrop" example that we were discussing earlier, one could very easily use the anchors + offset to position the tooltip on the end of the teardrop.

Another reason is that we can't currently express the Alignment idea in Wayland. Relying on enums lets the compositor do the right thing, regardless of what windowing layout scheme it has.

So our vote is leaving the enums for now, unless we use the API and see a good reason why we should switch to Alignment :) Let me know what you think! We can always come back to it if we decide it's not working for us

justinmc

LGTM 👍

I'm on board with the WindowPositionerAnchor enum instead of Alignment, especially given that it more closely maps to Waylands implementation. Thanks for all of the changes.

… positioning logic (flutter/flutter#177404)

…ning logic (flutter#177404) ## What's new? - Added tooltips to the windowing API, but they have no implementation for the time being - Added the `WindowPostiioner` logic to the windowing API - Wrote a lot of tests 🧪 ## Pre-launch Checklist - [x] I read the [Contributor Guide] and followed the process outlined there for submitting PRs. - [x] I read the [Tree Hygiene] wiki page, which explains my responsibilities. - [x] I read and followed the [Flutter Style Guide], including [Features we expect every widget to implement]. - [x] I signed the [CLA]. - [x] I listed at least one issue that this PR fixes in the description above. - [x] I updated/added relevant documentation (doc comments with `///`). - [x] I added new tests to check the change I am making, or this PR is [test-exempt]. - [x] I followed the [breaking change policy] and added [Data Driven Fixes] where supported. - [x] All existing and new tests are passing. # Conflicts: # packages/flutter/lib/src/widgets/_window_win32.dart

flutter/flutter@3f553f6...7b98d50 2025-11-26 98614782+auto-submit[bot]@users.noreply.github.com Reverts "Fix for win32 embedder failing to send all alt key downs to the flutter app (#179097)" (flutter/flutter#179136) 2025-11-26 [email protected] Fix for win32 embedder failing to send all alt key downs to the flutter app (flutter/flutter#179097) 2025-11-26 [email protected] Modernize framework lints (flutter/flutter#179089) 2025-11-25 98614782+auto-submit[bot]@users.noreply.github.com Reverts "Add framework-side hitTestBehavior support to Semantics (#178817)" (flutter/flutter#179100) 2025-11-25 [email protected] Add framework-side hitTestBehavior support to Semantics (flutter/flutter#178817) 2025-11-25 [email protected] Roll Packages from e019cf9 to cc3dca6 (1 revision) (flutter/flutter#179081) 2025-11-25 [email protected] Add tooltip windows to the windowing API alongside the window positioning logic (flutter/flutter#177404) 2025-11-25 [email protected] FlutterWindowsView::SendWindowMetrics now reliably sends the display_id (flutter/flutter#179053) 2025-11-25 [email protected] Remove semantics geometry shortcircuit (flutter/flutter#178680) 2025-11-25 [email protected] Add an assert message when OverlayEntry.remove is called twice (flutter/flutter#178163) 2025-11-25 [email protected] Roll Fuchsia Linux SDK from pOO9Jl9HTLsEmks6y... to nzuAxCJGeJbkZCTkr... (flutter/flutter#179066) 2025-11-25 [email protected] Dynamically set MinimumOSVersion in App.framework (flutter/flutter#178253) 2025-11-25 [email protected] Roll Skia from d83c30b090f4 to 925c311f4b37 (2 revisions) (flutter/flutter#179060) 2025-11-25 [email protected] Marks Linux build_android_host_app_with_module_aar to be unflaky (flutter/flutter#174864) 2025-11-25 [email protected] Marks Linux_mokey complex_layout__start_up to be unflaky (flutter/flutter#174865) 2025-11-25 [email protected] Manual Dart SDK roll to 3.11.0-169.0.dev (flutter/flutter#179054) 2025-11-25 [email protected] Bump Dart to 3.9 (flutter/flutter#179041) 2025-11-25 [email protected] Roll Skia from e298c2f93ebf to d83c30b090f4 (2 revisions) (flutter/flutter#179058) 2025-11-24 [email protected] updated licenses_cpp readme (flutter/flutter#178874) 2025-11-24 [email protected] Roll Skia from 43d2020be565 to e298c2f93ebf (5 revisions) (flutter/flutter#179046) 2025-11-24 [email protected] Refactor `_isLabel` method in `stepper.dart` to use `any` for better readablity (flutter/flutter#178909) 2025-11-24 49699333+dependabot[bot]@users.noreply.github.com Bump actions/checkout from 5 to 6 in the all-github-actions group (flutter/flutter#179049) 2025-11-24 [email protected] Disposes test restoration manager when accessed by bindings (flutter/flutter#176519) 2025-11-24 [email protected] [ Widget Preview ] Always generate scaffold under `$TMP` (flutter/flutter#179039) 2025-11-24 [email protected] Roll Packages from e67b6be to e019cf9 (9 revisions) (flutter/flutter#179035) 2025-11-24 [email protected] Update CHANGELOG.md for Flutter 3.38.3 (flutter/flutter#178935) 2025-11-24 [email protected] Remove unnecessary `String.valueOf` in `SettingsChannel.java‎` (flutter/flutter#178590) 2025-11-24 [email protected] Roll pub manually, pick up flutter_lints in examples/api (flutter/flutter#179030) 2025-11-24 [email protected] Roll Dart SDK from 24cc9a740bd3 to afca43095efa (1 revision) (flutter/flutter#179019) 2025-11-24 [email protected] Pass EXCLUDED_ARCHS from Xcode project to xcodebuild for macOS builds (flutter/flutter#176948) 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 [email protected],[email protected] 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

…ning logic (flutter#177404) ## What's new? - Added tooltips to the windowing API, but they have no implementation for the time being - Added the `WindowPostiioner` logic to the windowing API - Wrote a lot of tests 🧪 ## Pre-launch Checklist - [x] I read the [Contributor Guide] and followed the process outlined there for submitting PRs. - [x] I read the [Tree Hygiene] wiki page, which explains my responsibilities. - [x] I read and followed the [Flutter Style Guide], including [Features we expect every widget to implement]. - [x] I signed the [CLA]. - [x] I listed at least one issue that this PR fixes in the description above. - [x] I updated/added relevant documentation (doc comments with `///`). - [x] I added new tests to check the change I am making, or this PR is [test-exempt]. - [x] I followed the [breaking change policy] and added [Data Driven Fixes] where supported. - [x] All existing and new tests are passing.

… positioning logic (flutter/flutter#177404)

Add tooltip windows to the windowing API alongside the window positio…

e74d062

…ning logic

github-actions bot added the framework flutter/packages/flutter repository. See also f: labels. label Oct 22, 2025

mattkae requested review from justinmc, knopp, loic-sharma and robert-ancell October 22, 2025 19:04

gemini-code-assist bot reviewed Oct 22, 2025

View reviewed changes