Pressing back in setting goes back to home #47

Closed
opened 2022-08-28 22:14:50 +00:00 by davidoskky · 2 comments
Contributor

Steps to Reproduce

  1. Open the settings
  2. Open general settings
  3. Tap the back arrow on the upper left
  4. You're taken back to the home screen rather than the settings selection screen
### Steps to Reproduce 1. Open the settings 2. Open general settings 3. Tap the back arrow on the upper left 4. You're taken back to the home screen rather than the settings selection screen
Owner

This was introduced when I added this and this.

I needed to know when a user was coming back from the settings activity, and needed to add this, else, the onActivityResult code was not called.

This could be tricky.

This was introduced when I added [this](https://gitea.amine-louveau.fr/Louvorg/ReaderForSelfoss-multiplatform/src/branch/master/androidApp/src/main/java/bou/amine/apps/readerforselfossv2/android/HomeActivity.kt#L370) and [this](https://gitea.amine-louveau.fr/Louvorg/ReaderForSelfoss-multiplatform/src/branch/master/androidApp/src/main/java/bou/amine/apps/readerforselfossv2/android/HomeActivity.kt#L950). I needed to know when a user was coming back from the settings activity, and needed to add [this](https://gitea.amine-louveau.fr/Louvorg/ReaderForSelfoss-multiplatform/src/branch/master/androidApp/src/main/java/bou/amine/apps/readerforselfossv2/android/settings/SettingsActivity.kt#L220), else, the `onActivityResult` code was not called. This could be tricky.
Author
Contributor

Ok, I see the issue...
I'm currently busy restructuring a flat.
If I find some free time I'll take a look at that.

Ok, I see the issue... I'm currently busy restructuring a flat. If I find some free time I'll take a look at that.
Sign in to join this conversation.
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: Louvorg/ReaderForSelfoss-multiplatform#47
No description provided.