Closed Bug 84259 Opened 23 years ago Closed 23 years ago

Properties dialog: links should appear underlined [or, some sort of formatting] in modern

Categories

(SeaMonkey :: Themes, defect)

defect
Not set
normal

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: bugzilla, Assigned: hewitt)

References

Details

Attachments

(1 file)

couldn't find an existing a bug for this, but do dup and/or reassign as needed. basically, the links which appear in the Properties dialog [whether for links or images] are unformatted, ie, has the same appearance as the rest of the content in that dialog. perhaps the link should be underlined and/or colored? eg, perhaps use the same color/appearance styles as links in the browser?
It seems like the new modern are missing the "link" class I added to modern2 and classic when I did the properties window.
Any particular links? This WORKSFORME in 2001-05-25 Win95... Oh, I see. Jonas - do you want to sort this out? I'm rather tied up right now... Gerv
CC'ing hewitt since he's the creator of modern3. Hewitt, the new modern is missing the "link" class that I added when doing the properties window. The patches for that is avalible in bug 1995
Summary: Properties dialog: links should appear underlined [or, some sort of formatting] → Properties dialog: links should appear underlined [or, some sort of formatting] in modern
example: 1. go to http://faqs.org/ 2. bring up the Properties dialog for either the library image in the main frame, or the "no frames" image link in the left nav frame. yep, i see this with modern3, not with classic. [in classic the links are blue.] thx for pointing this out, jonas! should this go over to the themes folx?
IMHO it should. Assigning -> themes
Assignee: gervase.markham → hewitt
Component: XP Apps: GUI Features → Themes
QA Contact: sairuh → pmac
Keywords: modern
There has long been a class in the Modern and Classic themes called "text-link". Jonas, you should have used this in your XUL instead of "link". I'm not going to add the "link" class, since it is redundant. Please change your xul to use "text-link" and remove the "link" class from the places you added it in Classic.
Status: NEW → ASSIGNED
Keywords: modern
You tested it that it works, right? r=hwaara
sr=hewitt
a= asa@mozilla.org for checkin to the trunk. (on behalf of drivers)
Blocks: 83989
checked in
Status: ASSIGNED → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
Status: RESOLVED → VERIFIED
Verified the patch.
Product: Core → SeaMonkey
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: