[CLASSIC] File button (in Messenger) is wacked

VERIFIED FIXED

Status

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

People

(Reporter: Jerry Baker, Assigned: Joe Hewitt (gone))

Tracking

({classic})

Trunk
x86
Windows 2000
classic

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

(Reporter)

Description

18 years ago
Build: 2000100720

Open a mail folder or newsgroup in Mozilla with Classic skin. Make sure that 
when the group or folder is opened that no messages are selected. Mouseover the 
"File" button. The icon becomes un-disabled on mouseover, but no button outline.

Now select a message and the "File" button becomes enabled. Mouseover it 
again...no button outline still.

Comment 1

18 years ago
Sending to Joe, this may be related to another File button bug.
Assignee: hangas → hewitt
(Assignee)

Comment 2

18 years ago
I have a fix in hand for this.
Status: NEW → ASSIGNED
(Assignee)

Updated

17 years ago
Priority: P3 → P2
(Assignee)

Comment 3

17 years ago
I thought at first that this would be an easy fix, but after more careful
analysis I decided that the "right way" to fix this would be a bit more complicated.

The File button is actually a <button> within a <menu>.  This is kinda dumb
considering the existence of the <menubutton> tag.  So, my fix is to replace
this combo with a menubutton.  This required me to add some new XBL bindings and
style rules that weren't there before to handle the case of menubuttons that
should be styled like .button-toolbar-2 buttons.
(Assignee)

Comment 4

17 years ago
Created attachment 22310 [details] [diff] [review]
proposed patch
(Assignee)

Comment 5

17 years ago
Created attachment 22311 [details] [diff] [review]
proposed patch part 2

Comment 7

17 years ago
looks good, r=alecf
(Assignee)

Comment 8

17 years ago
fixed
Status: ASSIGNED → RESOLVED
Last Resolved: 17 years ago
Resolution: --- → FIXED

Comment 9

17 years ago
Marking verified on windows (2001-01-24-09-Mtrunk).
Status: RESOLVED → VERIFIED

Updated

17 years ago
Keywords: classic
(Reporter)

Comment 10

16 years ago
Mass removing self from CC list.
(Reporter)

Comment 11

16 years ago
Now I feel sumb because I have to add back. Sorry for the spam.
Product: Core → SeaMonkey
You need to log in before you can comment on or make changes to this bug.