With the Dusk skin, text enclosed in <a name=""></a> is displayed in blue

RESOLVED FIXED in Bugzilla 3.2



User Interface
11 years ago
11 years ago


(Reporter: Frédéric Buclin, Assigned: Wurblzap)


Bugzilla 3.2
Bug Flags:
approval +



(1 attachment, 1 obsolete attachment)

681 bytes, patch
Frédéric Buclin
: review+
Details | Diff | Splinter Review


11 years ago
Many places use <a name="foo">some text</a> to easily point to some portion of a page. But the Dusk skin displays all these strings in blue, as if they were real links. That's confusing and not really nice from a visual point of view. So we should only display text in blue if it's really a link. No idea how to do that in CSS (at least not from the top of my head).

Comment 1

11 years ago
You have to make your links <a name="something"></a>Text. You can't enclose the text in the anchor. This is a well-known issue that only shows up in Firefox.

Comment 2

11 years ago
(In reply to comment #1)
> You have to make your links <a name="something"></a>Text. You can't enclose the
> text in the anchor.

Of course you can, and is even recommended by the spec:

They even explicitly say that: "User agents should be able to find anchors created by empty A elements, but some fail to do so."

Comment 3

11 years ago
Created attachment 280162 [details] [diff] [review]
patch, v1

I use this trick for years on a website I'm managing.
Assignee: ui → LpSolit
Attachment #280162 - Flags: review?(wurblzap)

Comment 4

11 years ago
Comment on attachment 280162 [details] [diff] [review]
patch, v1

The underline on hover is removed on IE7, but it's still blue there. We may need some entries in IE-fixes.css.

It works on Firefox, so I'm willing to make this r+ if we agree to fix IE separately.
Attachment #280162 - Flags: review?(wurblzap) → review-

Comment 5

11 years ago
Marc, can you help with IE7? I have Linux and Windows 2000, so I cannot install IE7.

Comment 6

11 years ago
Created attachment 280245 [details] [diff] [review]
Patch 2

Taking bug as agreed on IRC.

Here's a patch which works on both IE7 and FF2. Frédéric, can you please test it on IE6?

On a side note, I'd like to understand why it's necessary to mention a:link:hover near the beginning of the patch. I'd've thought that this is covered by a:link, but it apparently isn't.
Assignee: LpSolit → wurblzap
Attachment #280162 - Attachment is obsolete: true
Attachment #280245 - Flags: review?(LpSolit)

Comment 7

11 years ago
Comment on attachment 280245 [details] [diff] [review]
Patch 2

Tested successfully with Firefox (Windows 2000 + Linux), Opera 9.23 (Linux), IE6 SP1 (Windows 2000) and Konqueror 3.5.6 (Linux). r=LpSolit
Attachment #280245 - Flags: review?(LpSolit) → review+


11 years ago
Flags: approval+

Comment 8

11 years ago
Checking in skins/contrib/Dusk/global.css;
/cvsroot/mozilla/webtools/bugzilla/skins/contrib/Dusk/global.css,v  <--  global.css
new revision: 1.2; previous revision: 1.1
Last Resolved: 11 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.