Closed Bug 46073 Opened 24 years ago Closed 24 years ago

Windows Classic Theme plain button error

Categories

(SeaMonkey :: Themes, defect, P3)

Product:
SeaMonkey
Component:
Themes
Platform:
x86
Windows 95
Type:
defect

Tracking

(Not tracked)

Status:
VERIFIED FIXED

People

(Reporter: neil, Assigned: hewitt)

Details

(Keywords: classic, Whiteboard: nsbeta3+) 

One of the styles that should be applied to plain buttons in
classic/global/style/button.css doesn't have the .plain class modifier and gets
applied to all active hovered buttons.
Line 113:
  button:hover:active > .button-internal-box,
should read:
  button.plain:hover:active > .button-internal-box,
as it does in modern/global/style/button.css
Additionally, line 120:
  button-plain > .button-internal-box > .button-icon,
should read:
  button.plain > .button-internal-box > .button-icon,
This particular error also appears to apply to the modern skin.
Sending to Brendan as Classic Skin Lead.
Assignee: hangas → bdonohoe
setting bug status to New
Status: UNCONFIRMED → NEW
Ever confirmed: true
Adding classic keyword
Keywords: classic
Reassigning from bdonohoe to hangas
Assignee: bdonohoe → hangas
Component: Themes → User Interface: Design Feedback
QA Contact: paw → mpt
--> Themes
Component: User Interface: Design Feedback → Themes
QA Contact: mpt → pmac
Marking nsbeta3+ sending to Joe
Assignee: hangas → hewitt
Keywords: nsbeta3
Whiteboard: nsbeta3+
Joe please check for this problem in Mac Classic and Modern skins.

this was already fixed in modern2 and blue, and is now fixed in mac/win/unix 
classic.  I'll check it in as soon as my cvs account is ready.
fixed as of a few days ago
Status: NEW → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
Joe, 
Would you show me how to reproduce this bug? It seems this bug I need
to look into the code.
Thanks,
patty
Looks OK now.
Status: RESOLVED → VERIFIED
Product: Core → SeaMonkey
You need to log in before you can comment on or make changes to this bug.