-
Notifications
You must be signed in to change notification settings - Fork 3.3k
[google_maps_flutter_web] Migrate to package:web #5254
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
Changes from all commits
8800cc7
0c9fc45
8960e3d
b420f32
a0637f1
636e71f
639fe97
c22678e
147e570
5ff2982
c4581dd
0fb3d67
9898737
18564d2
aa9533a
1667de9
c43f31e
a14981c
32d66fa
788d35a
1040103
edc00a8
cd2b0e8
861f294
bb8810f
e578d86
638a466
75db30d
2c086d2
fb7f53a
aee9770
9f2716e
cb9c665
aa46171
0523daa
e2df4b1
e358399
736aec1
8b4d7bb
d8adca1
52baeca
1951a6a
6583739
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -20,6 +20,9 @@ class MyApp extends StatefulWidget { | |
class _MyAppState extends State<MyApp> { | ||
@override | ||
Widget build(BuildContext context) { | ||
return const Text('Testing... Look at the console output for results!'); | ||
return const Directionality( | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I accidentally ran the example app once directly (using There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This is fine, the skeleton app is only used to have a Flutter context where things run, but IIRC this particular test does not render a map or anything. |
||
textDirection: TextDirection.ltr, | ||
child: Text('Testing... Look at the console output for results!'), | ||
); | ||
} | ||
} |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -3,21 +3,22 @@ publish_to: none | |
|
||
# Tests require flutter beta or greater to run. | ||
environment: | ||
sdk: ^3.1.0 | ||
flutter: ">=3.13.0" | ||
sdk: ^3.3.0 | ||
flutter: ">=3.19.0" | ||
|
||
dependencies: | ||
flutter: | ||
sdk: flutter | ||
google_maps_flutter_platform_interface: ^2.4.0 | ||
google_maps_flutter_web: | ||
path: ../ | ||
web: ^0.5.0 | ||
|
||
dev_dependencies: | ||
build_runner: ^2.1.1 | ||
flutter_test: | ||
sdk: flutter | ||
google_maps: ^6.1.0 | ||
google_maps: ^7.1.0 | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This is not blocking for this review, but when I tried to do a Wasm build of the google maps' example app for web, I still get some errors coming from the google_maps package:
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Do we file a tracking issue for google_maps WASM support? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I think we should, compiling to There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. /FYI @kevmoo! There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Ah 💩 . Ugh... There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. |
||
google_maps_flutter: ^2.2.0 # Needed for projection_test.dart | ||
http: ">=0.13.0 <2.0.0" | ||
integration_test: | ||
|
Uh oh!
There was an error while loading. Please reload this page.