Closed
Bug 53189
Opened 24 years ago
Closed 24 years ago
Print dialog opens up twice when clicked on Print button triangle in composition toolbar
Categories
(Core :: DOM: Editor, defect, P3)
Tracking
()
VERIFIED
FIXED
People
(Reporter: shrir, Assigned: bugzilla)
References
Details
Attachments
(1 file)
(deleted),
patch
|
Details | Diff | Splinter Review |
builds: 2000091808m18, all platforms
Steps:
1 Launch Composer
2 Click on the triangle below the Print button in COmposition Toolbar
3 Observe a Print dialog pop up.
4 Click on CANCEL to close it
5 Observe that another PRINT dialog pops up
Expected: Print dialog should not open up twice
Assignee | ||
Comment 2•24 years ago
|
||
This also happens in MailNews.
This seems to happen because the parent (the button) observes cmd_print and so
does it's child (the menupopup). This works fine if I just remove
observes="cmd_print" from the menupopup, since it's derived from the parent.
reassigning to me
Assignee: don → blakeross
Assignee | ||
Comment 3•24 years ago
|
||
Comment 4•24 years ago
|
||
Are those <menuitems> shared by any other buttons or main menus? Could this break
the print menu item?
Assignee | ||
Comment 5•24 years ago
|
||
no and no.
Comment 6•24 years ago
|
||
I believe this is because the oncommand event fires on the menuitem, and the
event bubbles up to the button, and fires there too. This fix seems reasonable
given that. a=ben.
Assignee | ||
Comment 7•24 years ago
|
||
Fix checked in to trunk.
Status: ASSIGNED → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
REOPENING...not fixed on branch or trunk.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Assignee | ||
Comment 10•24 years ago
|
||
This will not be fixed on the branch, it is not rtm++. It is working fine for
me on the trunk in all cases. Please doublecheck...
Comment 11•24 years ago
|
||
oops its fixed...my mistake..I was verifying the wrong trunk.
Status: REOPENED → RESOLVED
Closed: 24 years ago → 24 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 13•24 years ago
|
||
No problem. If you want this fixed on the branch, feel free to nominate...but
don't hold your breath.
Assignee | ||
Comment 14•24 years ago
|
||
*** Bug 57986 has been marked as a duplicate of this bug. ***
You need to log in
before you can comment on or make changes to this bug.
Description
•