-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Help button #1415
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
Help button #1415
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1415 +/- ##
=========================================
- Coverage 3.3% 3.18% -0.12%
=========================================
Files 128 132 +4
Lines 6839 7092 +253
Branches 669 682 +13
=========================================
Hits 226 226
- Misses 6598 6851 +253
Partials 15 15
Continue to review full report at Codecov.
|
|
IMO, we should keep Yes as the primary action and More Help as the secondary one. So, More Help can be just a link instead of a button. @neslihanturan @misaochan your thoughts? |
|
Sorry, just managed to look at this. I definitely agree that the "more help" action should be the secondary one. I would recommend a link in the bottom right corner instead. Additionally, perhaps "more information" might be better than "more help"? What do you think @neslihanturan ? |
|
Ok I'll make the changes |
|
Is top right better ? @misaochan |
|
I'd personally go with bottom right, but we can wait for Neslihan to weigh in. :) |
|
Ok :) |
|
I've initially pushed the top right version, will modify after further discussions :) |
|
Bottom right is better for me too @misaochan |
|
Okk I'll change that @neslihanturan |
|
@97balakrishnan were you able to make this change. We can go ahead and merge it once the UI is fixed. :) |
|
Sorry for the long delay |
|
Works as intended @97balakrishnan , thanks! |

Description
Fixes #1397
Added a button to open the wikimedia commons help page at the end of the tutorial
Tests performed
Tested on Android 6.0.1 Moto G3
Screenshots showing what changed