8000 Fix #2194: Updated Nearby ⓘ icon to include 'Search this area' feature by ShridharGoel · Pull Request #2198 · commons-app/apps-android-commons · GitHub
Skip to content

Fix #2194: Updated Nearby ⓘ icon to include 'Search this area' feature #2198

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

Merged
merged 1 commit into from
Mar 17, 2019

Conversation

ShridharGoel
Copy link
Contributor

Description (required)

Fixes #2194

What changes did you make and why?

Updated content of dialog box which opens up on clicking the i icon on the Nearby tab.

Tests performed (required)

Tested betaDebug on API 25.

Screenshots showing what changed

screenshot_2018-12-20-23-27-29-954_fr free nrw commons beta

@domdomegg domdomegg self-requested a review December 20, 2018 18:17
Copy link
Member

@domdomegg domdomegg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Because this is basically just wording and it was largely implemented by @neslihanturan, requesting her to review too.

@domdomegg domdomegg changed the title Updated Nearby i icon to include 'Search this area' feature Updated Nearby ⓘ icon to include 'Search this area' feature Dec 20, 2018
@commons-app commons-app deleted a comment from codecov-io Dec 20, 2018
@domdomegg domdomegg changed the title Updated Nearby ⓘ icon to include 'Search this area' feature Fix #2194: Updated Nearby ⓘ icon to include 'Search this area' feature Dec 20, 2018
@@ -296,7 +296,7 @@
<string name="retry">Retry</string>

<string name="showcase_view_got_it_button">Got it!</string>
<string name="showcase_view_whole_nearby_activity">These are the places near you that need pictures to illustrate their Wikipedia articles</string>
<string name="showcase_view_whole_nearby_activity">These are the places near you that need pictures to illustrate their Wikipedia articles.\n\nClicking on \'SEARCH THIS AREA\' locks the map and launches a nearby search around that location.</string>
<string name="showcase_view_list_icon">Tapping this button brings up a list of these places</string>
Copy link
Collaborator

@neslihanturan neslihanturan Dec 21, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hmm, I think locking the map is something only developers should know. But I am not sure what kind of message would be better. Let's wait some ideas from team.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What about "...fixes the map at that location..."? Or maybe we can omit that part and just write that it launches a nearby search around that location?

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lets discuss under #2194 so that everyone can follow.

@domdomegg
Copy link
Member

What's going on with this PR - have we agreed on some wording for this yet @neslihanturan?

@domdomegg
Copy link
Member

I'm going to merge this for now given it's been here so long, if we need to change the wording we can open a new issue.

@domdomegg domdomegg merged commit 0b25730 into commons-app:master Mar 17, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants