Skip to content

Commit

Permalink
Update browser warning text
Browse files Browse the repository at this point in the history
As advised by the content team.

- Lead with the action
- Don't use 'should'
  • Loading branch information
timpaul authored and bradwright committed Jun 10, 2014
1 parent 008a96a commit 62f4667
Showing 1 changed file with 2 additions and 3 deletions.
5 changes: 2 additions & 3 deletions app/assets/javascripts/browser-check.js
Expand Up @@ -7,11 +7,10 @@ $(function() {
"use strict";
function browserWarning() {
var container = $('<div id="global-browser-prompt"></div>'),
text = $('<p>For a safer, faster, better experience online you should upgrade your browser. </p>'),
findMoreLink = $('<a href="/help/browsers">Find out more about browsers</a>'),
text = $('<p><a href="/help/browsers">Upgrade your web browser</a> (the software you use to access the internet), it’s out of date</p>'),
closeLink = $('<a href="#" class="dismiss" title="Dismiss this message">Close</a>');

return container.append(text.append(findMoreLink, closeLink));
return container.append(text.append(closeLink));
}

// we don't show the message when the cookie warning is also there
Expand Down

1 comment on commit 62f4667

@phae
Copy link
Contributor

@phae phae commented on 62f4667 Sep 3, 2014

Choose a reason for hiding this comment

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

How come you guys changed the text here? The new text is much less friendly/informative, IMHO.

Please sign in to comment.