Skip to content

fix: Add fitsSystemWindows to handle edge-to-edge behavior in Android 15 #508

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

Merged
merged 5 commits into from
Feb 14, 2025

Conversation

FR3DD221
Copy link
Contributor

Changes

  • Added android:fitsSystemWindows="true" to handle system window insets properly
  • Ensures content is properly displayed without being obscured by the status bar

Testing

Verified on:

  • Android 15 (API 35) - Fixed
  • Android 7.1 (API 25) - No regression

@miltonials
Copy link
Member

Hi @FR3DD221 !! Thanks for the PR
Can you please put the android:fitsSystemWindows="true" to the others activities too? They has the same issue

@jamescr
Copy link
Member

jamescr commented Feb 10, 2025

This PR fix #505

fitsSystemWindows to displaytrackmap and gpsstatus.

WaypointsList.xml: Order the code and change fill-parent for match-parent, because fill is deprecated

WaypointsList.java: Added ListView configuration in onCreate to properly handle system insets and some visual properties

Preferences: Added ListView configuration in onCreate to properly handle system insets and some visual properties
Deleting a spanish comment y a some empty lines
@jamescr jamescr added this to the 2024.12 --> 2025.02 milestone Feb 13, 2025
This method was deleted by accident, so i´m adding it again.
Copy link
Member

@Andyporras Andyporras left a comment

Choose a reason for hiding this comment

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

I tested it on android 15 and it works fine.👌

Copy link
Member

@miltonials miltonials left a comment

Choose a reason for hiding this comment

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

I have tested this PR using API25 is working well!

Testing just the PR changes: everything works as expected. ✅
Merging this PR to #513: No issues found, all functionalities remain stable. ✅

@jamescr jamescr merged commit 88503ad into labexp:develop Feb 14, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants