Page MenuHomePhabricator

remove some CSS for abbr and acronym tags
Closed, ResolvedPublic

Description

Author: a.d.bergi

Description:
The <abbr> and <acronym> tags were whitelisted with bug 671, but there are some CSS rules for them since long, long times. They can be found in the first versions of chick, monobook and are carried on to vector skin.

The lines I'm complaining about are

abbr, acronym, .explain {
color: black;
background: none;
}

which are senseless from my perspective.

Often these tags are used in links, e.g. [[Normalnull|<abbr title="meter above see level">NN</abbr>]]. But in here the color:black; makes the text unrecognizable as a hyperlink (together with the senseful cursor:help; rule).

When these rules where meant to override some crazy browserdependent default settings, they should be be changed to "inherit".


Version: 1.17.x
Severity: major

Details

Reference
bz28979

Event Timeline

bzimport raised the priority of this task from to Needs Triage.Nov 21 2014, 11:31 PM
bzimport set Reference to bz28979.
bzimport added a subscriber: Unknown Object (MLST).

a.d.bergi wrote:

proposed patch

Attached:

As I recall they are indeed meant to override some crazy browser defaults; "inherit" had problems in some older browsers but is *probably* fine now.

Needs a confirmation check in supported browsers.

a.d.bergi wrote:

applied by Mark with r88118, but monobook was forgotten?