Conversation
There was a problem hiding this comment.
Pull request overview
This PR refactors the Nearby map UI layout to group the “recenter” and “legend” FloatingActionButtons into a single vertically-stacked container, simplifying positioning and spacing within the parent RelativeLayout.
Changes:
- Removed individually-positioned
fab_recenter/fab_legenddefinitions. - Reintroduced those FABs inside a new vertical
LinearLayoutaligned to the end of the map area. - Updated margins/spacing by moving common positioning to the container.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
✅ Generated APK variants! |
|
We have FABs nested inside a LinearLayout and not CoordinatorLayout explicitly used as mentioned in the PR description. Is this understanding correct? |
Yes, you are absolutely correct, and I have updated the PR description to reflect the use of LinearLayout. |
|
I'll test it in some time, although I'm expecting linear layout to fix the issue :) |
Sure , Thanks :) |
|
By the way, curious to know about the other approach you'd mentioned: did you try anchoring as well? |
It was not working properly hence the other approach |
Review by GSoC ApplicantI reproduced the issue from Before applying changes: Screen_Recording_20260319_215515_Commons.mp4I then pulled this PR branch ( After applying changes: Screen_Recording_20260319_215612_Commons.mp4What I checked:
The reported issue is resolved with this PR. ✅ |


Description (required)
Fixes #6699
Wrapped FABs in a vertical LinearLayout to fix icon distortion and alignment issues in landscape mode.
Tested 6.3.0-debug on Pixel 9 (Android 16)
Screenshots (for UI changes only)

