Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[Windows] [Multi-window] Share internal plugins #115611

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
loic-sharma opened this issue Nov 18, 2022 · 5 comments
Closed

[Windows] [Multi-window] Share internal plugins #115611

loic-sharma opened this issue Nov 18, 2022 · 5 comments
Labels
a: desktop Running on desktop a: multi window Issues related to multi window support P3 Issues that are less important to the Flutter project platform-windows Building on or for Windows specifically team-windows Owned by the Windows platform team

Comments

@loic-sharma
Copy link
Member

Background

On Windows, several internal plugins are owned by the view (FlutterWindowsView). These plugins are intended to be singletons and should be shared in a multi-view world.

Solution

Share internal plugins across views by making them owned by the engine.

@loic-sharma
Copy link
Member Author

The Windows embedder's plugins are now shared by the engine.

@github-project-automation github-project-automation bot moved this from In Progress to Done in Multi Window Feb 10, 2023
@github-actions
Copy link

github-actions bot commented Mar 3, 2023

This thread has been automatically locked since there has not been any recent activity after it was closed. If you are still experiencing a similar issue, please open a new bug, including the output of flutter doctor -v and a minimal reproduction of the issue.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 3, 2023
@loic-sharma loic-sharma reopened this Jul 13, 2023
@loic-sharma loic-sharma moved this from Done to Ready in Multi Window Jul 13, 2023
@flutter flutter unlocked this conversation Jul 13, 2023
@loic-sharma
Copy link
Member Author

Reopening as the text input plugin's initialization depends on view creation.

auto-submit bot pushed a commit to flutter/engine that referenced this issue Nov 9, 2023
Currently the keyboard tests use global state to track key calls. As a result, a good test can fail if a bad test leaves key calls in the global state. This removes the global state and ensures each test is isolated.

Preparation for flutter/flutter#115611

[C++, Objective-C, Java style guides]: https://github.com/flutter/engine/blob/main/CONTRIBUTING.md#style
auto-submit bot pushed a commit to flutter/engine that referenced this issue Nov 30, 2023
Currently the text input plugin is strongly tied to a single view. This change makes the text input plugin tied to the engine in preparation for multi-view world.

Part of flutter/flutter#115611

[C++, Objective-C, Java style guides]: https://github.com/flutter/engine/blob/main/CONTRIBUTING.md#style
@loic-sharma loic-sharma removed their assignment Mar 27, 2024
@loic-sharma
Copy link
Member Author

Addressed by flutter/engine#51758

@github-project-automation github-project-automation bot moved this from Ready to Done in Multi Window Apr 5, 2024
Copy link

This thread has been automatically locked since there has not been any recent activity after it was closed. If you are still experiencing a similar issue, please open a new bug, including the output of flutter doctor -v and a minimal reproduction of the issue.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Apr 19, 2024
@cbracken cbracken added team-windows Owned by the Windows platform team and removed team-desktop labels Jun 6, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
a: desktop Running on desktop a: multi window Issues related to multi window support P3 Issues that are less important to the Flutter project platform-windows Building on or for Windows specifically team-windows Owned by the Windows platform team
Projects
Status: Done
Development

No branches or pull requests

2 participants