Page MenuHomePhabricator

Different colours for visited/unvisited and :active external links (like internal links) (accessibility!)
Closed, ResolvedPublic

Description

Author: W

Description:
On en.wikipedia using the monobook skin we are allowed six different colors
for Wikilinks: (normal/stub/missing) * (unvisited/visited). But we are only
allowed one color for external links. Please allow our browsers to
discriminate between visited and unvisited external links.

These lines work for me (the first one is already in main.css):

#bodyContent a.external { color: #36B; }
#bodyContent a.external:visited { color: #749; }


Version: unspecified
Severity: normal
URL: http://en.wikipedia.org/skins-1.5/monobook/main.css

Details

Commits
Unknown Object (Diffusion Commit)
Reference
bz3112

Event Timeline

bzimport raised the priority of this task from to Normal.Nov 21 2014, 8:46 PM
bzimport set Reference to bz3112.

robchur wrote:

Customise this using custom CSS as desired.

jasonspiro4 wrote:

I've switched to the new Vector skin, but enwiki still doesn't differentiate between visited and unvisited external links by default.

(In reply to comment #1)

Customise this using custom CSS as desired.

It's nice that this is user-customizable. But most users don't install custom CSS, so it'd be very helpful if you set it by default.

At http://www.useit.com/alertbox/20040503.html usability expert Jakob Nielsen writes, "Knowing which pages they've already visited frees users from unintentionally revisiting the same pages over and over again." He says that this is extra-important for users with bad short-term memory, such as many older users, but important for everyone.

Reopening. Bumping from "trivial" to "normal" because this is a usability-related request, not a cosmetic issue. Adding keywords "accessibility, easy".

Would you accept a patch to fix this?

sorin.sbarnea wrote:

Anyone with usability/accessibility experience will tell you that :visited are required. Please repair it.

helloworld101 wrote:

Proposed patch that adds :visited styling

Patch for vector skin's main-ltr.css and main-rtl.css which adds :visited styling to external links so they are the same colour as visited internal links.

Attached:

Jorm added a comment.Sep 4 2010, 1:17 AM

The color #0b0080 is not web safe so will probably not be used. I'll see what I can do about finding a web-safe one.

(In reply to comment #5)

The color #0b0080 is not web safe so will probably not be used. I'll see what
I can do about finding a web-safe one.

Does web safe colours really matter anymore? I'd imagine most people can display all colours expressible in #rrggbb format now a days.

Jorm added a comment.Sep 9 2010, 4:27 PM

Websafe doesn't matter to 99.99999% of the web anymore, no.

But we are not within that superset.

Can you update your patch with a correct color and provide a patch for the differents skins ? :)

  • Bug 26475 has been marked as a duplicate of this bug. ***

wdwester wrote:

If the bug is marked as a duplicate then obviously it should be twice as important to resolve. The problem does not need to be solved by installing a file on my computer. This a problem for every visitor to the WIKI sites. The problem needs to solved by the programmers or so called web site builders for everyones benefit including a first time visitor. Whats wrong with plain old RED?

  • Bug 26475 has been marked as a duplicate of this bug. ***
Reedy added a comment.Feb 11 2011, 1:24 AM

CC'ing Trevor and Krinkle...

This is a very trivial bug, and by the looks of it, also occurs on Vector. Maybe other skins too....

Any chance of fixing this up and closing it?

I'll do this next monday when finishing up other small things.

Fixed in r82495.

In addition to the color for .external:visited, I added a color for .external:active ( 'active' is when your mouse is down on it). This to be consistent with internal links which also have a visual clue for :active.

external links that were interwikis (eg. [[m:Foobar]] ) had an override to match external links, for that reason I added declerations to have a visual clue for :active and :visted on .extiw as well.

Diffusion added a commit: Unknown Object (Diffusion Commit).Mar 4 2015, 8:16 AM