Windows Classic theme should not use ThreeDFace for borders

VERIFIED FIXED in mozilla0.9.6

Status

SeaMonkey
Themes
P5
normal
VERIFIED FIXED
18 years ago
10 years ago

People

(Reporter: neil@parkwaycc.co.uk, Assigned: shliang)

Tracking

Trunk
mozilla0.9.6
x86
Windows 95
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(2 attachments, 1 obsolete attachment)

(Reporter)

Description

18 years ago
All insets (and outsets) currently using threedface and threeddarkshadow should
all be using threedlightshadow instead of threedface.

Comment 1

18 years ago
Sending to Joe
Assignee: hangas → hewitt

Updated

18 years ago
Status: NEW → ASSIGNED

Comment 2

17 years ago
Don't think is gonna get approved for RTM.  Marking P5 future.
Priority: P3 → P5
Target Milestone: --- → Future

Comment 3

17 years ago
marking dup


*** This bug has been marked as a duplicate of 57429 ***
Status: ASSIGNED → RESOLVED
Last Resolved: 17 years ago
Resolution: --- → DUPLICATE

Comment 4

17 years ago
Verified dup.
Status: RESOLVED → VERIFIED
(Reporter)

Comment 5

17 years ago
ThreeDFace should only be used for control backgrounds.
Borders for focus effects should use transparent.
Borders for inset and outset effects should use ThreeDLightShadow.

Follow the URL to see a list of the offending styles.
Status: VERIFIED → REOPENED
Resolution: DUPLICATE → ---
Summary: Windows Classic theme should use threedlightshadow for insets → Windows Classic theme should not use ThreeDFace for borders

Updated

17 years ago
Status: REOPENED → ASSIGNED

Comment 6

16 years ago
Mass reassigning some of my theme bugs to Shuehan.
Assignee: hewitt → shliang
Status: ASSIGNED → NEW
(Assignee)

Updated

16 years ago
Status: NEW → ASSIGNED
Target Milestone: Future → mozilla0.9.6
(Assignee)

Comment 7

16 years ago
Created attachment 52594 [details] [diff] [review]
replaced some 3dface with 3dlightshadow
(Assignee)

Updated

16 years ago
Blocks: 105174
(Assignee)

Comment 8

16 years ago
Created attachment 55314 [details] [diff] [review]
doing this over
(Assignee)

Updated

16 years ago
Attachment #55314 - Attachment is obsolete: true
(Assignee)

Comment 9

16 years ago
Created attachment 55354 [details] [diff] [review]
doing this over

Comment 10

16 years ago
Comment on attachment 55354 [details] [diff] [review]
doing this over

r=jag
Attachment #55354 - Flags: review+

Comment 11

16 years ago
Comment on attachment 55354 [details] [diff] [review]
doing this over

sr=hewitt
Attachment #55354 - Flags: superreview+
(Assignee)

Comment 12

16 years ago
checked in
Status: ASSIGNED → RESOLVED
Last Resolved: 17 years ago16 years ago
Resolution: --- → FIXED

Comment 13

16 years ago
Verified the patch.
Status: RESOLVED → VERIFIED
(Reporter)

Updated

16 years ago
Blocks: 110254
Product: Core → SeaMonkey
You need to log in before you can comment on or make changes to this bug.