-
Notifications
You must be signed in to change notification settings - Fork 59
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 swipe to vote change throws you back to the root settings page (#810) #815
Conversation
- fix the rerender upon swipeToVote change by buffering the change first -add todo notices for missing translations
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This doesn't behave very well since if you leave the settings page before the invisible buffer takes effect, it doesn't apply the settings.
You mean like when you are "super" fast with the switch and then instantly switch pages? |
I didn't even try to be that fast, it was like pretty casual speed imo and still didn't take effect. I can record in a min. |
I believe you like that, don't worry. If you have an idea feel free to tell me cause the one with leaving the page seemed reasonable |
# Conflicts: # src/components/screens/Settings/Appearance/AppearanceScreen.tsx
It should now get applied properly. There is still the odd reload when changing its state and then clicking the back button in the header but I have no idea right now how to fix that as that settings affects the whole screen (all of them) due to that global gesture |
It was actually just that the whole stack was getting re-rendered on the change. I adjusted that here in this PR: #823 I copied your changes over for the locale stuff though, since I had forgotten to do that. |
PR Creator Checklist
Ensure you've checked the following before submitting your PR:
Summary
this fixes #810
Screenshots
Simulator.Screen.Recording.-.iPhone.14.-.2023-07-22.at.15.00.49.mp4
Test Plan
Settings
->Appearance
Feed
Settings
->Appearance
.Feed
-> Can't swipe anymore but now you can use the fullscreen "go back" gesture instead