Skip to content
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

Fix DividerItemDecoration duplication and GPX export issues (#479 #480) #481

Merged
merged 3 commits into from
Dec 12, 2024

Conversation

miltonials
Copy link

Hi! This PR addresses issues #479 and #480

Modifications


Finally, coded strings have been replaced by @string resources to improve localization and maintenance.

…torage

- Extracted permission handling logic into a reusable method.
- Added support for Android R+ to avoid unnecessary permission requests.
- Displayed a rationale dialog when permission is not granted.
@miltonials miltonials marked this pull request as ready for review December 9, 2024 05:16
@Binnette Binnette merged commit 76f5100 into labexp:develop Dec 12, 2024
1 of 2 checks passed
@Binnette
Copy link
Collaborator

Hi @miltonials can you make a need PR to propose better messages for :

  1. To export the GPX trace we need to write on the storage.
  2. To display the track properly we need access to the storage.
  3. To share the track properly we need access to the storage.
  4. To upload the track to OSM we need access to the storage.

In fact, I don't like the "we need". Instead should be "OSMTracker needs". Also can you please try to make shorter sentences ?

So maybe:

  1. To export the GPX trace, OSMTracker needs storage access.
  2. To display the track, OSMTracker needs storage access.
  3. To share the track, OSMTracker needs storage access.
  4. To upload to OSM, OSMTracker needs storage access.

Or maybe:

  1. OSMTracker needs storage access to export the GPX trace.
  2. OSMTracker needs storage access to display the track.
  3. OSMTracker needs storage access to share the track.
  4. OSMTracker needs storage access to upload to OSM.

Or even more simple, just one message "OSMTracker needs storage access to perform this action"

What do you think ? If you can please try/test to see what solution fit the best 🙏

Thanks a lot, for fixing these old strings 😉👍

@miltonials
Copy link
Author

Hi @Binnette! 👋🏼
Your proposal sounds good! I’ll test the options you suggested and see which one works best, aiming for clarity and brevity. I’ll also ensure the final messages fit well within the UI/UX

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