Skip to content

fix : alignment nearby list item#6713

Open
rovertrack wants to merge 1 commit intocommons-app:mainfrom
rovertrack:list
Open

fix : alignment nearby list item#6713
rovertrack wants to merge 1 commit intocommons-app:mainfrom
rovertrack:list

Conversation

@rovertrack
Copy link
Contributor

@rovertrack rovertrack commented Mar 7, 2026

Fixes alignment of items in nearby list view

What changes did you make and why?
added proper margin and padding to items

Screenshots (for UI changes only)
Before :

Screenshot_2026-03-07-19-12-09-83_d5db3f3edc380047609fe9c266f7c566

After :
Screenshot_2026-03-07-19-10-34-93_062c6066f629c24413cc8efde3bd9f38

@amahuli03
Copy link

amahuli03 commented Mar 12, 2026

Review by GSoC applicant

I was able to reproduce the issue by navigating to the nearby list:
Screenshot_20260312_125127
As you can see, the icons to the left are not aligned with each other or with the text.

After pulling @rovertrack's changes, I repeated the steps from earlier and could see the changes visually:
Screenshot_20260312_130105

The original issue (mis-aligned images and text in the nearby list) was successfully addressed with this PR!

I did find one issue with these changes after looking into how LinearLayout works. This PR changes the root of item_place.xml from a RelativeLayout to a LinearLayout, which stacks its children top to bottom on its own. RelativeLayout uses android:layout_below="@+id/icon" to position siblings. layout_below is an attribute of RelativeLayout.LayoutParams. Now, we use LinearLayout, which does not recognize layout_below and silently skips it.

To avoid confusion, you should remove android:layout_below="@+id/icon" entirely. I tested this, and the page renders the same (with the proper alignment we want). That line makes no difference since it is dead code.

@neeldoshii neeldoshii self-requested a review March 12, 2026 18:29
@rovertrack
Copy link
Contributor Author

rovertrack commented Mar 13, 2026

@amahuli03 thank you very much for the review .
the dead code was present from the main branch ( inside a linear layout ) , to keep the changes minimal i had not touched that thanks for pointing out

@github-actions
Copy link

✅ Generated APK variants!

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.

2 participants