[Bug]: Map pin does not center when location is changed and centering option is turned on

[ramsaptami]

Is there an existing issue for this?

• I have searched the existing issues

Current Behavior

Pinned location should move when location changes on the map when Map & marker centering is turned on. Currently, the pin moves as location changes but manual intervention is needed to center or even bring the pin into view since it does not auto-adjust.

Steps To Reproduce

1. Open up map widget with map & marker centering turned on and bind location using watchPosition() on a button trigger as seen in the video
2. Change user location using sensor
3. Observe the pin position move but map view remain the same - this needs to dynamically change as per setting

Environment

Release

Version

Cloud

[somangshu]

The functionality is actually to center the pointer on click. We have to think about another way to achieve this

@wmdev0808 @jsartisan any solution in mind?

[wmdev0808]

@somangshu ,
I've reviewed the issue and gave it a try.
Please assign this issue to me so I can make the commit on my solution

[ramsaptami]

This isn't working again

[somangshu]

The only change made here Approx 2 months ago was with #9714 (App theming) and #12962 (map centring issue).

Seems like the change in #12962 could be causing this. @techbhavin kindly verify this and do make sure that you add test case this time to avoid regression.

[techbhavin]

@ramsaptami @somangshu

As per my testing, I am not able to reproduce my issue. I attached the Below video for reference.
Let me know if I am following the wrong step.

Application Link: https://release.app.appsmith.com/app/common-testing-application/page-62be8b496d836615ef8a90ac

Screen.Recording.2022-07-01.at.1.32.32.PM.mov

[ramsaptami]

Seems like the issue appears when accessing lat and long values from stored value. Directly accessing it from the geolocation function works as expected

[techbhavin]

@somangshu
any update for me on this issue?

[techbhavin]

@ramsaptami can you plz share the step and application to reproduce this issue.

[ramsaptami]

Bind lat and long from Appsmith store values - something like {{appsmith.store.lat}} {{appsmith.store.long}}
Observe that the value is read as a string and not number