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)
SeaMonkey
Themes
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.
Comment 2•23 years ago
|
||
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
Reporter | ||
Comment 4•23 years ago
|
||
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
Assignee | ||
Comment 6•23 years ago
|
||
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
Comment 8•23 years ago
|
||
You tested it that it works, right? r=hwaara
Assignee | ||
Comment 9•23 years ago
|
||
sr=hewitt
Comment 10•23 years ago
|
||
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
Comment 12•23 years ago
|
||
Verified the patch.
Updated•16 years ago
|
Product: Core → SeaMonkey
You need to log in
before you can comment on or make changes to this bug.
Description
•