Skip to content
This repository was archived by the owner on Feb 22, 2023. It is now read-only.

Conversation

@cyanglaz
Copy link
Contributor

@cyanglaz cyanglaz commented Feb 5, 2019

A temporary workaround to fix the camera positioning issue mentioned in flutter/flutter#24806

@cyanglaz cyanglaz requested a review from amirh February 5, 2019 00:21

- (void)mapView:(GMSMapView*)mapView didChangeCameraPosition:(GMSCameraPosition*)position {
if (!_cameraDidInitialSetup) {
// A temporary workaround for a bug that the camera is not properly positioned at
Copy link
Contributor

Choose a reason for hiding this comment

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

Note that we suspect that this is an issue with the iOS Google Maps SDK(which is still being investigated), and that we call moveCamera to the current position as a temporary workaround.
Also include a TODO to remove this once the Maps SDK issue is resolved(I'm ok with closing the current Github issue with this workaround, and filing a separate one for this TODO).

Copy link
Contributor

Choose a reason for hiding this comment

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

I meant note it in the code comment 😄

Copy link
Contributor Author

Choose a reason for hiding this comment

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

updated the comments, and also opened the issue ticket.

Copy link
Contributor

@amirh amirh left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@@ -1,3 +1,7 @@
## 0.2.0+1

* Fixed a bug which the camera is not positioned correctly at map initialization(temporary workaround).
Copy link
Contributor

Choose a reason for hiding this comment

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

nit: we can reference the github issue.
I would also mentioned that it's a work around, e.g "Fixed(worked around) a bug..."

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I did add the workaround in the end of the sentence. Did you mean to remove it and add Fixed(worked around)?

Copy link
Contributor

Choose a reason for hiding this comment

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

ohh oops missed that, that looks good.

Copy link
Contributor

Choose a reason for hiding this comment

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

still lets reference the github issue

@cyanglaz cyanglaz merged commit f34a4e2 into flutter:master Feb 6, 2019
@cyanglaz cyanglaz deleted the mapview_workaround branch February 6, 2019 18:57
andreidiaconu pushed a commit to andreidiaconu/plugins that referenced this pull request Feb 17, 2019
…flutter#1157)

A temporary workaround to fix the camera positioning issue mentioned in [flutter/flutter#24806](flutter/flutter#24806)
A follow-up issue has been opened to track the 'real' fix.(flutter/flutter#27550)
andreidiaconu added a commit to andreidiaconu/plugins that referenced this pull request Feb 17, 2019
karantanwar pushed a commit to karantanwar/plugins that referenced this pull request Feb 19, 2019
…flutter#1157)

A temporary workaround to fix the camera positioning issue mentioned in [flutter/flutter#24806](flutter/flutter#24806)
A follow-up issue has been opened to track the 'real' fix.(flutter/flutter#27550)
karantanwar pushed a commit to karantanwar/plugins that referenced this pull request Feb 19, 2019
…flutter#1157)

A temporary workaround to fix the camera positioning issue mentioned in [flutter/flutter#24806](flutter/flutter#24806)
A follow-up issue has been opened to track the 'real' fix.(flutter/flutter#27550)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants