@UseExperimental should take multiple marker classes [145137892]

Fixed

Feature Request

Status Update

No update yet.

Description

tr...@google.com

created issue #1

Nov 25, 2019 07:01PM

The AndroidX UseExperimental annotation only takes a single markerClass. This means only a single Experimental API can be used within the same scope. Taking an array of marker classes may be one way to fix this behavior.

By comparison, the kotlin UseExperimental annotation takes a vararg markerClass argument which allows for multiple experimental APIs in the same scope.

Comments

tr...@google.com <tr...@google.com> Nov 26, 2019 12:44AM

Assigned to al...@google.com.

al...@google.com <al...@google.com> #2Dec 3, 2019 05:33PM

Actually, a multiplier of 8.67 gets me back to the approximate position it should be.

tr...@google.com <tr...@google.com> #3Dec 3, 2019 09:00PM

This bug was fixed in 2.1.1. Thanks for the report!

al...@google.com <al...@google.com> #4Dec 4, 2019 04:38PM

While the behavior has changed, it is still incorrect, IIUC. Now the view is shifted up too much for the values entered. For instance, when I enter a bottom inset of 200, which is less than half of the total height (509) of the map, and so should have no effect on the default centering behavior, the marker is actually centered at 354.5 points from the bottom, which is the center of the map plus half of the bottom inset. This seems incorrect per my understanding of that API.

al...@google.com <al...@google.com> Dec 6, 2019 08:21PM

Accepted by al...@google.com.

ap...@google.com <ap...@google.com> #5Dec 20, 2019 07:36PM

Marked as fixed.

354.5 sounds exactly right to me.

Issue 145137892