Skip to content
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

[#1505] Reword View Requests to Browse Requests in nav #1536

Merged
merged 2 commits into from
Aug 21, 2014

Conversation

garethrees
Copy link
Member

Fixes #1505.

User testing highlighted that it was easy to mistake "View Requests" for
a Facebook-style wall rather than the list of all requests. [1]

I've added a flag to only make this change for WDTK because translations,
but I'm not sure how easy it would be to test that the change is good or not,
so I'm thinking we just roll with "browse"?

[1] #1505

@crowbot
Copy link
Member

crowbot commented Jun 3, 2014

Once you click on 'Browse Requests", you see the text below it "View and search requests" - should this be changed to "Browse and search requests" ?

@@ -181,7 +181,7 @@ def list
end

@filters = params.merge(:latest_status => @view)
@title = _("View and search requests")
@title = _("Browse and search requests")

Choose a reason for hiding this comment

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

Prefer single-quoted strings when you don't need string interpolation or special symbols.

@garethrees
Copy link
Member Author

Marked as Awaiting Deploy but not merging until we actually make a release so that we remember to make the changes to the locale files (See comments on 3a001ec)

@garethrees garethrees changed the title Reword View Requests to Browse Requests in nav [#1505] Reword View Requests to Browse Requests in nav Jun 11, 2014
@mysociety-pusher mysociety-pusher force-pushed the issues/1505-nav-browse-requests branch from 3b74b50 to 8ecec52 Compare August 21, 2014 15:48
User testing highlighted that it was easy to mistake "View Requests" for
a Facebook-style wall rather than the list of all requests. [1]

[1] #1505
Also for "View and search requests" --> "Browse and search requests"
@mysociety-pusher mysociety-pusher force-pushed the issues/1505-nav-browse-requests branch from 8ecec52 to ebb2b78 Compare August 21, 2014 15:56
@mysociety-pusher mysociety-pusher merged commit ebb2b78 into rails-3-develop Aug 21, 2014
@mysociety-pusher mysociety-pusher deleted the issues/1505-nav-browse-requests branch August 28, 2014 15:43
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.

4 participants