Page MenuHomePhabricator

"plainlinks" class issues in IE, Opera
Closed, ResolvedPublic

Description

Author: mrnobo1024

Description:
from main.css:
#bodyContent .plainlinks a {

background: none !important;
padding: 0;

}
This will override the #bodyContent a[href ^="http://"]
rule that comes before it in main.css, but not the
#bodyContent a.external rule in the later included
IE60Fixes.css. The padding here needs !important too


Version: unspecified
Severity: normal
URL: http://test.leuksman.com/view/714

Details

Reference
bz714

Revisions and Commits

Event Timeline

bzimport raised the priority of this task from to Medium.Nov 21 2014, 6:59 PM
bzimport set Reference to bz714.
bzimport added a subscriber: Unknown Object (MLST).
  • Bug 1516 has been marked as a duplicate of this bug. ***

robchur wrote:

Patch against CVS HEAD

Should fix it.

Attached:

robchur wrote:

I have to admit to not having confirmed the bug prior to patching it...:$

gangleri wrote:

added some RTL examples at
http://test.leuksman.com/index.php?title=714&oldid=10701#conclusions

In one case IE shows the icon twice; this is bug ...

best regards reinhardt [[user:gangleri]]

robchur wrote:

Comment migrated from bug 4338:

"In the Opera web browser the "plainlinks" are not shown as they should. There
is a gap between the link and the following letter or full stop (the external
link symbol is only hidden, but not removed)."

robchur wrote:

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

Wiki.Melancholie wrote:

This has to be added to main.css:

#bodyContent .plainlinks a {padding: 0 !important}

.plainlinks a.external.text:after {display:

none !important}

robchur wrote:

Should now be fixed in CVS HEAD.

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