Skip to content
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

Resolve initial view annotation placement issue #1604

Conversation

ZiZasaurus
Copy link
Contributor

Removed the addition of view annotation to the containerView, which caused the view annotation to initially load in the top left corner.

@ZiZasaurus ZiZasaurus requested a review from a team as a code owner September 27, 2022 15:50
@maios
Copy link
Contributor

maios commented Sep 30, 2022

Some tests are failing, could you check that?
Also with this new implementation, the observers will be notified about annotation view's visibility changed when the view is first added, could we add a test case for this one and maybe a changelog entry also?

Copy link

@Deloww Deloww left a comment

Choose a reason for hiding this comment

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

@ZiZasaurus ZiZasaurus enabled auto-merge (squash) October 6, 2022 13:58
@ZiZasaurus ZiZasaurus merged commit 2e46db5 into main Oct 6, 2022
@ZiZasaurus ZiZasaurus deleted the MAPSIOS-339-next-door-annotation-views-render-at-generic-location-on-creation-before-jumping-to-proper-coordinates branch October 6, 2022 14:04
OdNairy pushed a commit that referenced this pull request Oct 7, 2022
* resolving initial annotation view placement issue
OdNairy pushed a commit that referenced this pull request Oct 7, 2022
* resolving initial annotation view placement issue
OdNairy added a commit that referenced this pull request Oct 7, 2022
* Update GL-Native and Common versions

* Update Maps version to 10.9.0-rc.1

* fix zh localization (#1602)

* Write static config only if directory exists (#1608)

* Write static config only if directory exists

* Fix MapboxMaps -> MapboxMaps-static

* Enable ASan, JUnit collection, fix e2e, store crashes (#1622)

* Store crashes during unit tests

* Use JUnit for unit tests

* Fix e2e script on pushes without linked PR

* Enable Address Sanitizer for tests

* Resolve issue where selected view annotation is not moved to correct z-order (#1607)

* fix issue where selected view annotation is not moved to correct z-order

* Resolve initial view annotation placement issue (#1604)

* resolving initial annotation view placement issue

* Smooth reduced accuracy ring radius interpolation (#1625)

Co-authored-by: Ankur Khandelwal <ankur.khandelwal@mapbox.com>
Co-authored-by: Mai Mai <mai.mai@mapbox.com>
Co-authored-by: ZiZi <44972592+ZiZasaurus@users.noreply.github.com>
Co-authored-by: Roman Laitarenko <roman.laitarenko@mapbox.com>
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.

4 participants