Last modified: 2014-11-17 10:36:05 UTC
Since we already store is-redirect info for each entry in cur, adding a visual feature / distinction to identify links that link to redirects, with the same query that tests for existence, should not impose a performance hit. It would help catch inappropriate redirects, and reduce the number of redirs alltogether.
This bug does not depend on bug 17; removing dependency.
There is already 4 colours for links (visisted|unvisited) + (existent|nonexistent). When redirect to an article one see a notice like: "redirected from [[redirection article]]" That should be enough to fix redirects :o)
I'm going to reopen this, and say that while they should not be visually identified, they should be identifi*able*, with an appropriate CSS class as we do for new/extiw/external/stub etc. The number of requests we have to do this in individual places clearly demonstrates the desirability of marking them universally, and now that we have a usable and cannonical redirect table, this should be fairly simple to implement in Linker::link(). It sounds like this was attempted in bug 12968, but I guess not completely?
*** Bug 4709 has been marked as a duplicate of this bug. ***
*** Bug 12968 has been marked as a duplicate of this bug. ***
Class is already added universally to all links... except when using linkKnown().
I think this bug can be easily addressed. We use this mechanism on special:Allpages to differ between pages and redirects, so can be done the same for the rest of the pages.