-
Notifications
You must be signed in to change notification settings - Fork 804
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
Add a custom back button to the account wizard's advanced setup page #4686
Conversation
Codecov Report
@@ Coverage Diff @@
## master #4686 +/- ##
==========================================
- Coverage 56.43% 56.42% -0.01%
==========================================
Files 138 138
Lines 17068 17069 +1
==========================================
Hits 9632 9632
- Misses 7436 7437 +1
|
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.
Looks great! I would suggest removing the arrow <
Edit: This is there is all the back buttons, so this is not an issue :)
A question: Is it not possible to navigate back to the page where you enter your server address? This navigates to the first setup screen
Agree with @nimishavijay's feedback. Adding:
|
744a082
to
b6bc885
Compare
Ok, I renamed the button to "Cancel": |
Signed-off-by: Claudio Cambra <claudio.cambra@gmail.com>
b6bc885
to
5e67405
Compare
AppImage file: Nextcloud-PR-4686-5e67405b45a0bc33f4c27bc03915a48c083815cc-x86_64.AppImage |
Kudos, SonarCloud Quality Gate passed! |
/backport to stable-3.5 |
This PR adds a back button to the advanced setup page of the accounts wizard that returns the user to the beginning of the wizard.
Addresses a point in #4052
Screen.Recording.2022-06-30.at.14.27.58.mov
Signed-off-by: Claudio Cambra claudio.cambra@gmail.com