Skip to content

[Windows] Document the show window migration#9269

Merged
parlough merged 2 commits intoflutter:mainfrom
loic-sharma:windows_show_window_migration
Aug 19, 2023
Merged

[Windows] Document the show window migration#9269
parlough merged 2 commits intoflutter:mainfrom
loic-sharma:windows_show_window_migration

Conversation

@loic-sharma
Copy link
Member

@loic-sharma loic-sharma commented Aug 18, 2023

Flutter 3.7 and 3.10 has a race condition that can result in the window not being shown (see flutter/flutter#119415). Projects need to be migrated to fix this bug.

Most Windows projects will be migrated automatically by running flutter windows build or flutter run unless they've done significant changes to their windows/runner/flutter_window.cpp file.

This documents how to migrate a Windows project manually. Note that this fix depends on an API that was added in Flutter 3.13. In other words, projects that are still using Flutter 3.7 or 3.10 should not do this migration.

Issues fixed by this PR (if any): flutter/flutter#127695

image

Presubmit checklist

Copy link
Member

@parlough parlough left a comment

Choose a reason for hiding this comment

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

Thanks so much for staying on top of this, writing the doc, and the extra details! I really appreciate it :D

As a reference for next time, consider submitting the migration guide once the necessary changes land. These guidelines can still be helpful for users who follow beta. If something changes before stable is released, we can always readjust :)

@parlough parlough merged commit 8f490ce into flutter:main Aug 19, 2023
@loic-sharma loic-sharma deleted the windows_show_window_migration branch August 21, 2023 17:12
loic-sharma added a commit to flutter/gallery that referenced this pull request Sep 1, 2023
Migrate the Gallery app to ensure the window is shown.

Part of flutter/flutter#127695
Used by flutter/website#9269
genx7up pushed a commit to aweps/gallery that referenced this pull request Dec 13, 2023
Migrate the Gallery app to ensure the window is shown.

Part of flutter/flutter#127695
Used by flutter/website#9269
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.

2 participants