The default bug view has changed. See this FAQ.

New app menu doesn't have EDIT menu

RESOLVED FIXED in Thunderbird 21.0

Status

Thunderbird
General
RESOLVED FIXED
5 years ago
4 years ago

People

(Reporter: Mikeyy, Assigned: Paenglab)

Tracking

17 Branch
Thunderbird 21.0
x86_64
Windows 7
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments, 3 obsolete attachments)

(Reporter)

Description

5 years ago
I was doing Litmus test 10: Summary for collapsed threads - One thread, in which you have instructions: click a single message, then Edit | Select | Thread

Guess what, there is no Edit menu in new app menu button. :)
(Reporter)

Updated

5 years ago
Blocks: 789883
(Reporter)

Comment 1

5 years ago
My bad, there is EDIT menu item in new app menu, but it's grayed out, can't click on it.
I can open it by clicking on arrow right of EDIT, but it doesn't contain all options old EDIT menu contained.
(Assignee)

Comment 2

5 years ago
The Edit menu is splitted in Edit and Find menus to be consistent to Firefox. 'Favorite Folder' and 'Folder Properties' are in Folder menu where, I think, they are better located.

But you are right I missed 'Delete Message', 'Select Thread' and 'Starred Messages'.

Comment 3

5 years ago
(In reply to Mihovil Stanic [:Mikeyy - L10n HR] from comment #1)
> My bad, there is EDIT menu item in new app menu, but it's grayed out, can't
> click on it.

The "Edit" menu of the app button shows gray but shouldn't, its "Select All" item is selectable and active. That's probably a bug on its own.
(Reporter)

Comment 4

5 years ago
Select All is working as it should.

Comment 5

5 years ago
Yes, my point was that the "Edit" menu shouldn't appear disabled if at least one item in its submenu is enabled (and "Select All" usually is).
(Assignee)

Comment 6

4 years ago
Created attachment 692574 [details] [diff] [review]
proposed fix

I've added the 'Delete Message' and 'Select' menu to the 'Message' menu. I've also moved the 'Select All' from 'Edit' splitmenu to 'Select' menu to have all this entries together.

Blake, what do you think, should this go to TB 17 or is it okay to land for the public in TB 24?
Assignee: nobody → richard.marti
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Attachment #692574 - Flags: ui-review?(bwinton)
Attachment #692574 - Flags: review?(bwinton)
(Assignee)

Comment 7

4 years ago
Mac try build with this patch applied: http://ftp.mozilla.org/pub/mozilla.org/thunderbird/try-builds/richard.marti@gmail.com-9da0349d8598
Comment on attachment 692574 [details] [diff] [review]
proposed fix

So, I think this might be one of the places where we need to differ from Firefox, since we have the concept of messages as well as pages, and going to the Message menu to select all the threads seems really odd.

Having the single delete menu item also breaks up the message menu more than I would like.

So, given all that, I think I would prefer to move those items back into the edit menu, if possible.

(The code looks fine, though.  ;)

Thanks,
Blake.
Attachment #692574 - Flags: ui-review?(bwinton)
Attachment #692574 - Flags: ui-review-
Attachment #692574 - Flags: review?(bwinton)
Attachment #692574 - Flags: review+
(Assignee)

Comment 9

4 years ago
Blake, do you mean to remove the Edit splitmenu and add a normal Edit menu, without the cut/copy/paste buttons, above the File menu or put the menuitems into the Edit splitmenu?
Flags: needinfo?(bwinton)
I meant put the menuitems into the Edit splitmenu.
Flags: needinfo?(bwinton)
(Assignee)

Comment 11

4 years ago
Created attachment 696966 [details] [diff] [review]
All in splitmenu popup
Attachment #692574 - Attachment is obsolete: true
Attachment #696966 - Flags: ui-review?(bwinton)
Attachment #696966 - Flags: review?(bwinton)
(Assignee)

Comment 12

4 years ago
Created attachment 696967 [details]
Edit menu with patch

For easier ui-r
Comment on attachment 696966 [details] [diff] [review]
All in splitmenu popup

ui-r-wise, it's close, but there are a couple of changes I want to see.

I think we should have the Delete menu use "⌫" instead of the word "Backspace", at least on Mac.
  (I tried to see how that was done in the regular menu, but couldn't find it…)

I also think that we should either put all the items in the first sub-menu (for ease of access), and name them:
  Select All
  Select Thread
  Select Starred Messages
or leave them in the sub-menu, and name them:
  All
  Thread
  Starred Messages
I would prefer the first option, I think.

The code itself is fine, though, so r=me.

Thanks, and I apologize for the delay,
Blake.
Attachment #696966 - Flags: ui-review?(bwinton)
Attachment #696966 - Flags: ui-review-
Attachment #696966 - Flags: review?(bwinton)
Attachment #696966 - Flags: review+
(Assignee)

Comment 14

4 years ago
Created attachment 709778 [details] [diff] [review]
patch v2

(In reply to Blake Winton (:bwinton) from comment #13)
> Comment on attachment 696966 [details] [diff] [review]
> All in splitmenu popup
> 
> ui-r-wise, it's close, but there are a couple of changes I want to see.
> 
> I think we should have the Delete menu use "⌫" instead of the word
> "Backspace", at least on Mac.
>   (I tried to see how that was done in the regular menu, but couldn't find
> it…)

I also haven't found how it is made (maybe made by OS). I fixed it with defining the acceltext instead of giving the key under OSX. I hope this hack is okay. My tests worked with this.

> I also think that we should either put all the items in the first sub-menu
> (for ease of access), and name them:
>   Select All
>   Select Thread
>   Select Starred Messages
> or leave them in the sub-menu, and name them:
>   All
>   Thread
>   Starred Messages
> I would prefer the first option, I think.

I implemented the first option.
Attachment #696966 - Attachment is obsolete: true
Attachment #709778 - Flags: ui-review?(bwinton)
Attachment #709778 - Flags: review?(bwinton)
(Assignee)

Comment 15

4 years ago
Created attachment 709779 [details]
menu with patch v2 on OSX
Attachment #696967 - Attachment is obsolete: true
Comment on attachment 709778 [details] [diff] [review]
patch v2

Okay, ui-r=me, but I'ld like you to file a couple of followup bugs, if you don't mind.
1) The cut/copy/paste icons don't have a hover state, and I think they should act more like the ▶ icon.
2) The "Edit" text is always disabled, but I think it, too, should act like the ▶ icon.  (i.e. be enabled, and open the sub-menu when clicked.)

(If you wanted to fix either or both of those in this bug, I wouldn't object…  ;)

And the code itself seems fine, so I'll say r=me, too!

Thanks,
Blake.
Attachment #709778 - Flags: ui-review?(bwinton)
Attachment #709778 - Flags: ui-review+
Attachment #709778 - Flags: review?(bwinton)
Attachment #709778 - Flags: review+
(Assignee)

Comment 17

4 years ago
(In reply to Blake Winton (:bwinton) from comment #16)
> Comment on attachment 709778 [details] [diff] [review]
> patch v2
> 
> Okay, ui-r=me, but I'ld like you to file a couple of followup bugs, if you
> don't mind.
> 1) The cut/copy/paste icons don't have a hover state, and I think they
> should act more like the ▶ icon.

On Windows and Linux where is feedback because this are buttons. OSX has no feedback on hover -> I'll file a bug for OSX.

> 2) The "Edit" text is always disabled, but I think it, too, should act like
> the ▶ icon.  (i.e. be enabled, and open the sub-menu when clicked.)

Firefox uses this also only as label. I also don't know if this would be obvious when you hover the label on the left and a sub-menu opens on the right. And in between is a area with buttons and doesn't activate a sub-menu.
Keywords: checkin-needed
https://hg.mozilla.org/comm-central/rev/a2e5286e3d53
Status: ASSIGNED → RESOLVED
Last Resolved: 4 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 21.0

Comment 19

4 years ago
(In reply to Richard Marti [:Paenglab] from comment #17)
> > 2) The "Edit" text is always disabled, but I think it, too, should act like
> > the ▶ icon.  (i.e. be enabled, and open the sub-menu when clicked.)
> 
> Firefox uses this also only as label. I also don't know if this would be
> obvious when you hover the label on the left and a sub-menu opens on the
> right. And in between is a area with buttons and doesn't activate a sub-menu.

This is bug 834014 for Firefox and definitely an irritating behavior. "Edit" is the only menu with a submenu where the text is italic(?) and which is disabled even though it can be selected.

When you hover over the Edit's arrow to open the submenu, then move the mouse left and then down, the submenu stays open. The submenu of another item may open on top of it (not always reproducible, but frequently enough). So, there seems to be something wrong worthy of a bug report, the question is if you want to wait for a fix in the Firefox bug or investigate for Thunderbird in parallel.
(Assignee)

Updated

4 years ago
Blocks: 844432
You need to log in before you can comment on or make changes to this bug.