Permalink
Browse files

Fix Bug 28979 — “remove some CSS for abbr and acronym tags”

    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.

    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".

from Bergi
  • Loading branch information...
Mark A. Hershberger
Mark A. Hershberger committed May 14, 2011
1 parent cf20e03 commit 9433a9ede03a88de09171cce72c3f80170b5036e
Showing with 8 additions and 16 deletions.
  1. +0 −6 skins/chick/main.css
  2. +6 −0 skins/common/shared.css
  3. +2 −10 skins/vector/screen.css
@@ -146,12 +146,6 @@ select {
color: black;
vertical-align: top;
}
abbr, acronym, .explain {
border-bottom: 1px dotted black;
color: black;
background: none;
cursor: help;
}
q {
font-family: Times, "Times New Roman", serif;
font-style: italic;
@@ -4,6 +4,12 @@
* another, but don't ignore the poor pre-Monobook users either.
*/
/* Default style for semantic tags */
abbr, acronym, .explain {
border-bottom: 1px dotted black;
cursor: help;
}
/* Colored watchlist and recent changes numbers */
.mw-plusminus-pos { color: #006400; } /* dark green */
.mw-plusminus-neg { color: #8b0000; } /* dark red */
@@ -765,16 +765,8 @@ p {
margin: .4em 0 .5em 0;
line-height: 1.5em;
}
p img {
margin: 0;
}
abbr,
acronym,
.explain {
border-bottom: 1px dotted black;
color: black;
background: none;
cursor: help;
p img {
margin: 0;
}
q {
font-family: Times, "Times New Roman", serif;

0 comments on commit 9433a9e

Please sign in to comment.