Skip to content

Conversation

firebovine
Copy link

Please make sure these boxes are checked before submitting your pull request - thanks!

  • Acknowledge that you're contributing your code under Apache v2.0 license

  • Apply the AndroidStyle.xml style template to your code in Android Studio.

@firebovine firebovine changed the title Resolves #670 Show value of configuration in preferences for coordinate pref Feb 15, 2024
@barbeau
Copy link
Owner

barbeau commented Feb 16, 2024

Thanks @firebovine! I'll review this soon.

Copy link

stale bot commented Mar 23, 2025

This issue has been automatically marked as stale because it has not had recent activity. Please comment here if it is still valid so that we can reprioritize. Thank you!

Copy link
Owner

@barbeau barbeau left a comment

Choose a reason for hiding this comment

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

Thanks @firebovine! Sorry for the delayed review.

changePreferenceSummary(key)
} else {
Copy link
Owner

Choose a reason for hiding this comment

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

Can you please make this an else if?

@@ -277,6 +292,18 @@ class Preferences : PreferenceActivity(), OnSharedPreferenceChangeListener {
preferredSpeedUnits!!.summary = entries[i]
}
}
} else if (prefKey.equals(
Copy link
Owner

Choose a reason for hiding this comment

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

The formatting for this seems off - can you please reformat this file?

@stale stale bot removed stale labels Mar 27, 2025
@barbeau
Copy link
Owner

barbeau commented Mar 27, 2025

Could you also please resolve the conflicts?

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