-
-
Notifications
You must be signed in to change notification settings - Fork 373
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
Welcome screen gets displayed in weird situations #310
Comments
Thanks for the bug report! It really amazes me that there are so many unique ways to break my keyboard :) Tbh, I didn't even think of the fact that deleting the app data still lets you into the Settings without prior being on the setup screen when using the Settings shortcut from within the keyboard itself. This is due to the fact that when you press the FlorisBoard icon in the home screen, you are actually always launching the SetupActivity and when you've already completed the setup you'll automatically redirected to the SettingsMainActivity. When using the Settings shortcut from within the keyboard, you go immediately to the SettingsMainActivity. This bug is also reproducible after a clean install:
I will place this bug on my todo list to be fixed. |
It's pretty fun finding this stuff. :) So the solution is just to always show the welcome-screen upon first app launch? I suggest that you maybe skip the "Activate FlorisBoard" page if it already is active. |
I'll just do the same as with the home screen launcher: Always open the
I think letting it stay like it is now (that the success text is immediately there even though you didn't go to the system settings because FlorisBoard is already enabled). |
Above commit fixes the bug, it now always correctly shows the welcome screen if needed. Thanks again for reporting! |
Steps to reproduce
This basically completely reset the app, with the important difference being that the keyboard was still activated in the android-settings.
Nr. 4 in combination with Nr. 8 makes me think this isn't something that should have happened.
I'm not sure what fix would make the most sense though.
Urgency
Fixing this shouldn't be a priority I think. It should only happen with users that do weird stuff (like I did).
Word suggestions and #227 (and others) are way more important.
Environment information
The text was updated successfully, but these errors were encountered: