Created attachment 727340 [details] [diff] [review]
It's way illogical for the Attachments menu to be under File. It should of course be under the Message menu. Seamonkey fixed this years ago in bug 352696.
(See bug 352696 discussion, if you wonder why it's a little more changes than one would think.)
Created attachment 727354 [details] [diff] [review]
proposed fix, v2
Comment on attachment 727354 [details] [diff] [review]
proposed fix, v2
So, this mostly seems good to me, but…
>@@ -982,23 +982,16 @@
>- <menu id="appmenu_fileAttachmentMenu"
>- <menupopup id="appmenu_attachmentMenuList"
> <menuseparator class="appmenu-menuseparator"/>
>@@ -1920,16 +1913,23 @@
> <menuseparator id="appmenu_messageAfterOpenMsgSeparator"/>
>+ <menu id="appmenu_msgAttachmentMenu"
>+ <menupopup id="appmenu_attachmentMenuList"
>+ <menuseparator id="appmenu_messageAfterAttachmentMenuSeparator"/>
I think that the attachments for a message are important enough to leave in the top level, so let's revert this change.
(Or, well, let's revert part of this change. I'm happy to change the id from "appmenu_fileAttachmentMenu" to "appmenu_msgAttachmentMenu". ;)
(Reverting this also makes the two parts of the menu be the same length, which is kind of nice…)
r=me, and ui-r=me with that bit reverted.
Created attachment 732000 [details]
> I think that the attachments for a message are important enough to leave in the top level,
Well, for a *message* ;) On the other hand i don't know who (except possibly keyboard only users) would access accachments from the menus, as there are way more natural access points. It's in no way primary.
> Reverting this also makes the two parts of the menu be the same length
The "parts" are still same length, see the attached screenshot. If this was part of the descision, please reconsider. Otherwise, reconsider anyway, as having it in different places creates a bit of inconsistency.
It's not exactly in different places, since we already hoist a variety of things to that left-hand menu. I would even venture to say that it'll be less surprising to users, since I'm asking us to not move the attachments out of the left-hand menu, but instead to leave them where they are. ;)
And the larger gap at the bottom in the screenshot you posted seems uglier to me than having the attachments menu where it used to be.
I understand your objection, and have thought about it, but I stand by my original request.
http://hg.mozilla.org/comm-central/rev/68687817c22d -> FIXED