Enable swiping on comments to go back to posts#785
Conversation
twizmwazin
left a comment
There was a problem hiding this comment.
This is an improvement, but I have two thoughts:
- This should be enabled by default, honestly I don't even know if it needs an option. Do other apps offer this option?
- The animation for this when I tested it in the emulator seems to be fade-out/fade-in. Other apps have a finger tracking animation. That's what I imagined in #431. I'm not opposed to this since this is still an improvement, but I'd leave the original issue open until we can do a finger-tracking animation.
Fair enough. I wanted it to be less invasive as a first PR from me. As you said though I don't think there is any app that has it as an option. I will remove.
I'll see if I can make this happen tonight. I have not done much modern Android dev at all, I am primarily a web dev so I may not have the skills for this, but ill take a crack at it. |
|
@twizmwazin Alright I have updated it to have finger tracking animation. It would be nice to have the swipe background be the previous Activity & not be black, but I don't really know how I could make that happen. I could probably change it to be the users background theme color though if we want.
|
|
Yeah, |
|
Alright I updated it to use MaterialTheme.colorScheme.background on swipe. As for not having it be a solid color and be the HomeActivity I will probably need to add it to some app drawer or something? I don't know yet I would need to spend more time on it. Which I don't think I will get to for a couple days minimum. Edit:
sounds good I will circle back to this to improve it another time |
|
Themed swipe back
|
The Reddit clients I have always used on Android were Slide, Sync, Infinity, all of these by default support swiping left to right on a Post to go backwards. I have made code changes to make this possible on the comments to go back to the post view.
Closes #431
(Sorry for the laggy recording)