Menu "Message > Attachments" broken
Categories
(Thunderbird :: Mail Window Front End, defect)
Tracking
(thunderbird65 fixed, thunderbird66 fixed)
People
(Reporter: jpmengual, Assigned: alta88)
References
Details
(Keywords: regression)
Attachments
(1 file, 1 obsolete file)
4.07 KB,
patch
|
jorgk-bmo
:
review+
jorgk-bmo
:
approval-comm-beta+
|
Details | Diff | Splinter Review |
User Agent: Mozilla/5.0 (X11; Linux x86_64; rv:66.0) Gecko/20100101 Firefox/66.0
Steps to reproduce:
From a received message with an attach file:
- Press alt-m or f10 then right arrow eky until Message
- Press down key until "Attach file".
Actual results:
Here, the focus moves to the right menu instead of opening a submenu. Enter does notghing.
Expected results:
Right key or Enter should open a submenu displaying the file list, with themselves submenus to open, save, detach, etc.
Comment 1•6 years ago
|
||
I think something got lost in translation here. Sure, you can use keyboard navigation. The menu item is called "Attachments".
So here are some simpler STR:
- Have a message with an attachment.
- Tools > Attachments. This menu item stopped working.
Alice, can you please find the regression for us. Magnus/Arshad, do you feel responsible? Some de-XBL breakage? Already broken in TB 65 beta :-(
Comment 2•6 years ago
|
||
- Tools > Attachments. This menu item stopped working.
STR:
Select a message with an attachment in thread pane
Choose "Message" in Menu bar > "Attachments"
Actual results:
No submenu pops up
Regression window:
https://hg.mozilla.org/comm-central/pushloghtml?fromchange=9c6c4038b3b5ab074c1d9c2bf91bae0a062adaab&tochange=89435f04afee556e91d8f17634cd377dff5ac8d2
https://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=b4aeb99d1cb601e5a5288ca05630913fa8528a1c&tochange=6862624e24d005fb4f8fb07c6800d2acef1d287e
Comment 3•6 years ago
|
||
Oops, overlay removal caused this. Thank you so much as always, Alice!!
Comment 4•6 years ago
|
||
mail/base/content/mainNavigationToolbox.inc.xul
698 <menupopup id="attachmentMenuList" onpopupshowing="FillAttachmentListPopup(this);"/>
missing 1st arguments?
FillAttachmentListPopup( event , this)
Comment 5•6 years ago
|
||
Hmm, yes, I found two call sites where the event
argument was missing. Looking in the TB 60 code, it was missing there, too, but the overlay fixed it.
However, there's something else wrong, I marked the spot in the code.
Aceman, can you fix this for us please. Perhaps too much JS for Richard.
the overlay changes also forgot the menuitems.. (noticed and fixed this in the attachments sec patch).
- <menupopup id="attachmentMenuList" onpopupshowing="FillAttachmentListPopup(this);"/>
+ <menupopup id="attachmentMenuList"
+ onpopupshowing="FillAttachmentListPopup(event, this);">
+ <menuseparator/>
+ <menuitem label="&openAllAttachmentsCmd.label;"
+ accesskey="&openAllAttachmentsCmd.accesskey;"
+ command="cmd_openAllAttachments" />
+ <menuitem label="&saveAllAttachmentsCmd.label;"
+ accesskey="&saveAllAttachmentsCmd.accesskey;"
+ command="cmd_saveAllAttachments"/>
+ <menuitem label="&detachAllAttachmentsCmd.label;"
+ accesskey="&detachAllAttachmentsCmd.accesskey;"
+ command="cmd_detachAllAttachments" />
+ <menuitem label="&deleteAllAttachmentsCmd.label;"
+ accesskey="&deleteAllAttachmentsCmd.accesskey;"
+ command="cmd_deleteAllAttachments" />
+ </menupopup>
</menu>
<menuseparator id="messageAfterAttachmentMenuSeparator"/>
and appmenu
<menupopup id="appmenu_attachmentMenuList"
- onpopupshowing="FillAttachmentListPopup(this);"/>
+ onpopupshowing="FillAttachmentListPopup(event, this);">
+ <menuseparator/>
+ <menuitem label="&openAllAttachmentsCmd.label;"
+ accesskey="&openAllAttachmentsCmd.accesskey;"
+ command="cmd_openAllAttachments" />
+ <menuitem label="&saveAllAttachmentsCmd.label;"
+ accesskey="&saveAllAttachmentsCmd.accesskey;"
+ command="cmd_saveAllAttachments"/>
+ <menuitem label="&detachAllAttachmentsCmd.label;"
+ accesskey="&detachAllAttachmentsCmd.accesskey;"
+ command="cmd_detachAllAttachments" />
+ <menuitem label="&deleteAllAttachmentsCmd.label;"
+ accesskey="&deleteAllAttachmentsCmd.accesskey;"
+ command="cmd_deleteAllAttachments" />
+ </menupopup>
</menu>
<menuseparator class="appmenu-menuseparator"/>
Updated•6 years ago
|
Comment 8•6 years ago
|
||
You have to rebase your other stuff now ;-)
Pushed by mozilla@jorgk.com:
https://hg.mozilla.org/comm-central/rev/bb6728bfe435
Restore Attachments submenu which got lost during overlay removal. r=jorgk
Updated•6 years ago
|
Updated•6 years ago
|
Comment 10•6 years ago
|
||
TB 65 beta 3:
https://hg.mozilla.org/releases/comm-beta/rev/c9e0cf17a0df6ef6fba09e888f6f11135d877319
Reporter | ||
Comment 11•6 years ago
|
||
Hi,
I confirm the bug is fixed. Many thanks.
Regards
Updated•6 years ago
|
Description
•