See
http://www.w3.org/QA/Tips/noClickHere
Some text needs to be cleaned up to avoid "here" messages (after string 4.1 freeze).
Examples:
- templates/messages.tmpl: merge_notice
- templates/messages.tmpl: no_such_web
- templates/oopslanguagechanged.tmpl
- templates/oopslanguagechanged.pattern.tmpl
- templates/oopsaccessdenied.* (see Item3539)
There are probably more, also in topic text.
--
PTh
As I understand the advice, it is recommending against "click here" messages in normal web pages. Wiki pages are not normal web pages; readers
expect to see clickable functions, and printing out a page with an embedded "click here" is not really an issue. Are you
sure the above messages really need to be "cleaned up"?
CC
Definite yes.
Jakob Nielsen's Alertbox, Top Ten Web Design Mistakes of 2005: "Explain what users will find at the other end of the link, and include some of the key information-carrying terms in the anchor text itself to enhance scannability and search engine optimization (SEO). Don't use "click here" or other non-descriptive link text." (
http://www.useit.com/alertbox/designmistakes.html
)
--
PTh
OK, the search engine point is a good one. There really needs to be a "manual" for skin developers that covers these sorts of points.
CC
simple shanges like 'click [_2]' (where [_2] is the url) would be good - then you inform them where you're taking them too
-- SD
No, this is missing the point. Instead of:
Click here to get German beer if you are thursty.
write:
Get German beer if you are thursty.
which describes the action taken.
--
PTh
See also
Item3539
CC
Similarly, the many "OK"s in oops messages could be replaced by more meaningful links.
AC
Item3539 was marked as duplicate of this item, without updating this topic here. Done now at the top.
--
PTh
looks like the following need updating:
PatternSkin,
MailPageAddOn,
TopicClassificationAddOn,
NatSkin,
EmbedPDFPlugin, and
templates/messages.tmpl
,
templates/oopslanguagechanged.tmpl
, and
configure
in the core.
[go]$ grep -ri "click here" .
./twikiplugins/PatternSkin/templates/oopslanguagechanged.pattern.tmpl:%MAKETEXT{"Your language will be changed to [_1] from the next screen. You will be redirected to there right now. If you aren't, ~[~[[_2]~]~[click here~]~] to proceed." args="*%SESSION_VARIABLE{LANGUAGE}%*, %SCRIPTURLPATH{"view"}%/%WEB%/%TOPIC%"}%%TMPL:END%
./twikiplugins/MailPageAddOn/bin/mailpage/mailpage.cgi: <BR><font size="1"><a href="$ENV{'SCRIPT_NAME'}?action=privacy" target="_new">Click here</a> to read our privacy policy</font>
./twikiplugins/MailPageAddOn/bin/mailpage/readme.html:For the latest version of the script, <a href="#download">click here.</a><br>
./twikiplugins/MailPageAddOn/templates/mailpage.pattern.tmpl: <br /><font size="1"><a href="%SCRIPTURL{"mailpage"}%/mailpage.cgi?action=privacy" target="_new">Click here</a> to read our privacy policy</font>
./twikiplugins/MailPageAddOn/templates/mailpage.tmpl: <br /><font size="1"><a href="%SCRIPTURL{"mailpage"}%/mailpage.cgi?action=privacy" target="_new">Click here</a> to read our privacy policy</font>
./twikiplugins/TopicClassificationAddOn/data/TCAO/JustBlogItInstallation.txt:To get the !JustBlogIt extension, you can go to its [[http://blog.warmbrain.com/justblogit/][home page]], or if you're using a Mozilla/Firefird browser and are ready to install it, just [[http://blog.warmbrain.com/justblogit/justblogit-0.2.xpi][click here]] to install it.
./twikiplugins/NatSkin/data/TWiki/NatSkin.txt: * Test if installed: [[%SCRIPTURLPATH{"view"}%/%WEB%/%TOPIC%?skin=nat&style=Kubrick][click here]]
./twikiplugins/NatSkin/templates/messages.nat.tmpl:%MAKETEXT{"If you want to create the topic then [[[_1]][click here]]." args="%SCRIPTURLPATH{\"edit\"}%/%WEB%/%TOPIC%?t=%SERVERTIME{\"$year$mo$day$min$sec\"}%"}%
./twikiplugins/NatSkin/templates/oopslanguagechanged.nat.tmpl:%MAKETEXT{"Your language will be reset to the global default from the next screen on. You will be redirected to there right now. If you aren't, ~[~[[_1]~]~[click here~]~] to proceed." args="%SCRIPTURLPATH{"view"}%/%WEB%/%TOPIC%"}%
./twikiplugins/NatSkin/templates/oopslanguagechanged.nat.tmpl:%MAKETEXT{"Your language will be changed to [_1] from the next screen. You will be redirected to there right now. If you aren't, ~[~[[_2]~]~[click here~]~] to proceed." args="*%URLPARAM{language}%*, %SCRIPTURLPATH{"view"}%/%WEB%/%TOPIC%"}%
./twikiplugins/EmbedPDFPlugin/data/TWiki/EmbedPDFPlugin.txt: * Set LINKTEXT = (click here to display this PDF file by itself)
./templates/messages.tmpl:[[%SCRIPTURLPATH{"edit"}%/%WEB%/%TOPIC%?t=%SERVERTIME{"$year$mo$day$min$sec"}%][%MAKETEXT{"Click here"}%]].
./templates/oopslanguagechanged.tmpl:%MAKETEXT{"Your language will be changed to [_1] from the next screen. You will be redirected to there right now. If you aren't, ~[~[[_2]~]~[click here~]~] to proceed." args="*%SESSION_VARIABLE{LANGUAGE}%*, %SCRIPTURLPATH{"view"}%/%WEB%/%TOPIC%"}%
./lib/TWiki/Configure/Checkers/CGISetup.pm:<strong>Click here to test this</strong></a>
./lib/TWiki/Configure/UIs/PLUGINS.pm:Click here to consult the online plugins repository for
i see no need to change the test cases, and any ocurrances of "click here" in translation files should just "fall out" during translation after the main files have been updated.
./locale/pt.po:msgid "Click here"
./locale/pt.po:"redirected to there right now. If you aren't, [[%2][click here]] to proceed."
./locale/pt.po:#~ msgid "If you want to create the topic then [[%1][click here]]."
./locale/da.po:msgid "Click here"
./locale/da.po:"redirected to there right now. If you aren't, [[%2][click here]] to proceed."
./locale/cs.po:msgid "Click here"
./locale/cs.po:"redirected to there right now. If you aren't, [[%2][click here]] to proceed."
./locale/ru.po:msgid "Click here"
./locale/ru.po:msgid "Your language will be changed to %1 from the next screen. You will be redirected to there right now. If you aren't, [[%2][click here]] to proceed."
./locale/es.po:msgid "Click here"
./locale/es.po:"redirected to there right now. If you aren't, [[%2][click here]] to proceed."
./locale/es.po:#~ msgid "If you want to create the topic then [[%1][click here]]."
./locale/fr.po:msgid "Click here"
./locale/fr.po:"redirected to there right now. If you aren't, [[%2][click here]] to proceed."
./locale/fr.po:#~ msgid "If you want to create the topic then [[%1][click here]]."
./locale/de.po:msgid "Click here"
./locale/de.po:"redirected to there right now. If you aren't, [[%2][click here]] to proceed."
./locale/de.po:#~ msgid "If you want to create the topic then [[%1][click here]]."
./locale/sv.po:msgid "Click here"
./locale/sv.po:"redirected to there right now. If you aren't, [[%2][click here]] to proceed."
./locale/sv.po:#~ msgid "If you want to create the topic then [[%1][click here]]."
./locale/jp.po:msgid "Click here"
./locale/jp.po:msgid "Your language will be changed to %1 from the next screen. You will be redirected to there right now. If you aren't, [[%2][click here]] to proceed."
./locale/jp.po:#~ msgid "If you want to create the topic then [[%1][click here]]."
./locale/nl.po:msgid "Click here"
./locale/nl.po:"redirected to there right now. If you aren't, [[%2][click here]] to proceed."
./locale/nl.po:#~ msgid "If you want to create the topic then [[%1][click here]]."
./locale/zh-tw.po:msgid "Click here"
./locale/zh-tw.po:msgid "Your language will be changed to %1 from the next screen. You will be redirected to there right now. If you aren't, [[%2][click here]] to proceed."
./locale/zh-tw.po:#~ msgid "If you want to create the topic then [[%1][click here]]."
./locale/TWiki.pot:msgid "Click here"
./locale/TWiki.pot:"redirected to there right now. If you aren't, [[%2][click here]] to proceed."
./locale/pl.po:msgid "Click here"
./locale/pl.po:"redirected to there right now. If you aren't, [[%2][click here]] to proceed."
./locale/pl.po:#~ msgid "If you want to create the topic then [[%1][click here]]."
./locale/it.po:msgid "Click here"
./locale/it.po:"redirected to there right now. If you aren't, [[%2][click here]] to proceed."
./locale/it.po:#~ msgid "If you want to create the topic then [[%1][click here]]."
./locale/zh-cn.po:msgid "Click here"
./locale/zh-cn.po:msgid "Your language will be changed to %1 from the next screen. You will be redirected to there right now. If you aren't, [[%2][click here]] to proceed."
./locale/zh-cn.po:#~ msgid "If you want to create the topic then [[%1][click here]]."
./data/TestCases/TestCaseEmbeddedTags.txt:<a href="%SCRIPTURLPATH%/view/%WEB%/%TOPIC%?tagname=passed&searchtgt="ArfleBarfleGloop">Click here to run test</a>
./data/TestCases/TestCaseEmbeddedTags.txt:<a href="%SCRIPTURLPATH%/view/%WEB%/%TOPIC%?tagname=passed&passed=expected">Click here to run test</a>
./data/TestCases/TestCaseSaveContentWithoutEdit.txt:[[%SCRIPTURL{"save"}%/TestCases?topic=SavedContentExistingTopic&Rubaduba=%ENCODE{"%GMTIME%"}%][Click here]] to run the test. Expected result:
./data/TestCases/TestCaseSaveContentWithoutEdit.txt:[[%SCRIPTURL{"save"}%/TestCases?topic=SavedNewTopicNoEdit&templatetopic=TopicTemplateWithUrlParamFixture&testtext=text%20from%20the%20url&Rubaduba=%ENCODE{"%GMTIME%"}%][Click here]] to run the test and topic. Expected result:
./data/TestCases/TestCaseSaveContentWithoutEdit.txt:[[%SCRIPTURL{"save"}%/TestCases?topic=SavedNewTopicNoEdit&formtemplate=ThisForm&text=text%20from%20the%20url&Rubaduba=%ENCODE{"%GMTIME%"}%&Squeegle=Hugh][Click here]] to run the test and topic. Expected result:
./data/TestCases/TestCaseCreateNewTopicWithTemplateWithUrlParam.txt:[[%SCRIPTURL{"edit"}%/TestCases?topic=TemplatedTopic&templatetopic=TopicTemplateWithUrlParamFixture&testtext=text%20from%20the%20url][Click here]] to run the test and topic. *DO NOT SAVE IT*, just check that the content is present and correct.
./data/TestCases/TestCaseCreateNewTopicWithTemplate.txt: 1 [[%SCRIPTURL{"edit"}%/TestCases?topic=TemplatedTopic&templatetopic=TopicTemplateFixture][Click here]] to create the topic *DO NOT SAVE IT*!!
./data/TestCases/TestCaseCreateNewTopicWithTemplate.txt: 1 [[%SCRIPTURL{"edit"}%/TestCases/TemplatedTopic?templatetopic=TopicTemplateFixture][Click here]] to create the topic *DO NOT SAVE IT*!!
./data/TestCases/TestCaseCreateNewTopicWithTemplate.txt: 1 [[%SCRIPTURL{"edit"}%/TestCases/TemplatedTopic?templatetopic=TopicTemplateFixture&onlynewtopic=1][Click here]] to create the topic *DO NOT SAVE IT*!!
./data/TestCases/TestCaseAmISane.txt:| =configure= | The most basic sanity check | <a href="%SCRIPTURLPATH%/configure">click here</a> |
./data/TestCases/TestCaseAmISane.txt:| =changes= | Should redirect to changes | <a href="%SCRIPTURLPATH%/changes%SCRIPTSUFFIX%/%WEB%/TestCaseQuickScriptsChecks">click here</a> |
./data/TestCases/TestCaseAmISane.txt:| =edit= | Simple edit (save to create) | <a href="%SCRIPTURLPATH%/edit%SCRIPTSUFFIX%/%WEB%/QuickScriptsCheck?onlynewtopic=1&text=TEST%20PASSED%20-please%20save%20the%20topic">click here</a> |
./data/TestCases/TestCaseAmISane.txt:| =edit= | Simple edit (don't save) | <a href="%SCRIPTURLPATH%/edit%SCRIPTSUFFIX%/%WEB%/QuickScriptsCheck">click here</a> |
./data/TestCases/TestCaseAmISane.txt:| =preview= | Should redirect to preview | <a href="%SCRIPTURLPATH%/preview%SCRIPTSUFFIX%/TestCases/WasteOfSpace?text=Cabbages%20And%20Kings%20WalrusMeat&topicparent=TestCaseQuickScriptsChecks">click here</a> |
./data/TestCases/TestCaseAmISane.txt:| =rdiff= | Compare WebHome 1.1 and 1.2 sequential | <a href="%SCRIPTURLPATH%/rdiff%SCRIPTSUFFIX%/TestCases/WebHome?rev1=1&rev2=2">click here</a> |
./data/TestCases/TestCaseAmISane.txt:| =rdiff= | Compare WebHome 1.1 and 1.2 side by side | <a href="%SCRIPTURLPATH%/rdiff%SCRIPTSUFFIX%/TestCases/WebHome?rev1=1&rev2=2&render=sidebyside">click here</a> |
./data/TestCases/TestCaseAmISane.txt:| =save= | =action= = =save= Should redirect to view | <a href="%SCRIPTURLPATH%/save%SCRIPTSUFFIX%/TestCases/WasteOfSpace?action=save&text=Save%20PASSED&topicparent=TestCaseQuickScriptsChecks&forcerev=1">click here</a> |
./data/TestCases/TestCaseAmISane.txt:| =save= | =action= = =checkpoint= (should come back to edit) | <a href="%SCRIPTURLPATH%/save%SCRIPTSUFFIX%/TestCases/WasteOfSpace?action=checkpoint&text=Checkpoint-save%20PASSED&topicparent=TestCaseQuickScriptsChecks&forcerev=1">click here</a> |
./data/TestCases/TestCaseAmISane.txt:| =save= | =action= = =quietsave= (should redirect to view) | <a href="%SCRIPTURLPATH%/save%SCRIPTSUFFIX%/TestCases/WasteOfSpace?action=quietsave&text=Quietsave%20test%20PASSED&topicparent=TestCaseQuickScriptsChecks&forcerev=1">click here </a> |
./data/TestCases/TestCaseAmISane.txt:| =repRev= | =action= = =quietsave=, =cmd= = =repRev= if you are in Main.TWikiAdminGroup should redirect to view, and you should see "repRev test PASSED", otherwise you will be denied. | <a href="%SCRIPTURLPATH%/save%SCRIPTSUFFIX%/TestCases/WasteOfSpace?action=quietsave&cmd=repRev&text=RepRev%20test%20PASSED&topicparent=TestCaseQuickScriptsChecks&forcerev=1">click here </a> |
./data/TestCases/TestCaseAmISane.txt:| =delRev= | =action= = =save=, =cmd= = =delRev= - should delete the top revision, if you are in Main.TWikiAdminGroup, otherwise you will be denied. | <a href="%SCRIPTURLPATH%/save%SCRIPTSUFFIX%/TestCases/WasteOfSpace?action=quietsave&cmd=delRev&text=delRev%20test%20FAILED&topicparent=TestCaseQuickScriptsChecks&forcerev=1">click here </a> |
./data/TestCases/TestCaseAmISane.txt:| =search= | Search for the word "sanity" |<a href="%SCRIPTURLPATH%/search%SCRIPTSUFFIX%/TestCases/WasteOfSpace?search=sanity">click here</a> |
./data/TestCases/TestCaseAmISane.txt:| =statistics= | Should redirect to refreshed statistics |<a href="%SCRIPTURLPATH%/statistics%SCRIPTSUFFIX%/TestCases">click here</a> |
./data/TestCases/TestCaseAmISane.txt:| =attach=, =upload= | Attach a file to topic WasteOfSpace | <a href="%SCRIPTURLPATH%/attach%SCRIPTSUFFIX%/%WEB%/WasteOfSpace">click here</a> |
./data/TestCases/TestCaseAmISane.txt:| =rename= | Rename topic WasteOfSpace. Rename it within this web, and take care to clear the checkboxes so this topic doesn't get munged. | <a href="%SCRIPTURLPATH%/rename%SCRIPTSUFFIX%/TestCases/WasteOfSpace">click here</a> |
./data/TestCases/TestCaseAmISane.txt:| =rename= | Delete topic QuickScriptsCheck, checking refs within the current web _only_. | <a href="%SCRIPTURLPATH%/rename%SCRIPTSUFFIX%/TestCases/QuickScriptsCheck?newweb=Trash&nonwikiword=on¤twebonly=on">click here</a> |
./data/TestCases/TestCaseAmISane.txt:| =view= | Create non-existant (should redirect to create screen, which is an =oops= test) | <a href="%SCRIPTURLPATH%/view%SCRIPTSUFFIX%/%WEB%/DontCreateThisTopic">click here</a> |
--
WN - 20 May 2007
Reclassifying as Documentation, because it is, really.
CC