-
Notifications
You must be signed in to change notification settings - Fork 27.2k
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
Add basic web device and run support #28302
Conversation
For windows, I believe the command is as simple as |
friendly ping @yjbanov |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
|
||
Future<void> ensureReadyForPlatformSpecificTooling() async { | ||
/// Generate index.html in build/web. Eventually we could support | ||
/// a custom html under the web sub directory. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
FWIW internally we already switched to hand-written index.html
. We consider it equivalent of Java/ObjC code generated by flutter create
then edited by hand.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Makes sense, we can generate the index.html only as a part of flutter create and then separate out the generatedPluginRegistry files.
Description
Adds a skeleton web device, web specific project, and support for run via
--release --no-hot
. This device exists behind the FLUTTER_WEB env variable to prevent breaking default flutter run until we decide on the UX there. Added ability to launch chrome on mac.Related Issues
Fixes #28288
Fixes #28287
Part of #28285
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes (
[x]
). This will ensure a smooth and quick review process.///
).flutter analyze --flutter-repo
) does not report any problems on my PR.Breaking Change
Does your PR require Flutter developers to manually update their apps to accommodate your change?