279001 - Control-A and ctrl-T affect view setting when view field is focused - ctrl is ignored / hotkeys also fail

Status: RESOLVED FIXED

(Thunderbird :: Mail Window Front End, defect)

Description

[Robert Harvey]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.7.5) Gecko/20041107 Firefox/1.0
Build Identifier: Thunderbird version 1.0 (20041206)

A difficult one to describe, I'll try the step by step box.  My pop3 server was
recently reset by the provider, so all my stored messages were fetched again by
Thunderbird leading to lots of duplicates.

In an attempt to remove the duplicates I used 'view unread' to only display the
new versions, then used ^A and the delete icon to trash them.  Then when I
switched back to 'view all' the inbox was empty - all the contents of the inbox
had been moved.

Reproducible: Always

Steps to Reproduce:
1.  Have some unread messages in inbox
2.  Select 'view unread'
3.  select all 
4.  delete

Actual Results:  
every message from the inbox was moved to the trash folder, including the ones
previously read and not displayed.

Expected Results:  
only moved the unread messages to trash, because those were the only ones on
view and hence susceptible to selection.

Comment 1

[David :Bienvenu]

confirming and taking - that's ugly. I wonder if quick search has the same
problems (I don't think it does)

Comment 2

[Mike Cowperthwaite]

Interestingly, if I use the same MailView, but instead of Delete, I mark the 
selected messages with a Flag, only the ones that are visible get flagged.

Comment 3

[Tal Cohen]

This bug could well cause data-loss, if people are not aware of the "extra" 
deleted messages. I think it should be marked as "blocking".

Comment 4

[Robert Harvey]

Has this been fixed yet?

Comment 5

[Mike Cowperthwaite]

While the focus is *on* the mailview, the 'A' is grabbed without regard to 
the Ctrl modifier; and the 'A' is interpreted as the first letter in the 
"All" MailView.

Workarounds are: move the focus to the thread pane before typing Ctrl-A, 
or use Edit|Select All, or use the shift-clicking with the mouse to select.

This is Seamonkey bug 75587.

Comment 6

[Mike Cowperthwaite]

*** Bug 315725 has been marked as a duplicate of this bug. ***

Comment 7

[Wayne Mery (:wsmwk)]

more pervasive than just control-A

Comment 8

[Mike Cowperthwaite]

*** Bug 317896 has been marked as a duplicate of this bug. ***

Comment 9

[Lee_Dailey]

howdy y'all,

my tb info ...
Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.8) Gecko/20051201 Thunderbird/1.5 - Build ID: 2005120115

this is NOT limited to just trunk. happens with new, clean install of tb150 final with a new, clean profile. lookee ...

- TB1.5 bug? When View:Unread is selected and Ctrl+A is hit...
http://forums.mozillazine.org/viewtopic.php?t=370571

take care,
lee

Comment 10

[Vasil Dinkov]

(In reply to comment #3)
> This bug could well cause data-loss, if people are not aware of the "extra" 
> deleted messages. I think it should be marked as "blocking".

I absolutely agree! This bug is potentially very dangerous.

It is even worse if people have set TB to empty their Trash folder on exit. Imagine what will happen if they have moved all their messages from the Inbox folder to the Trash without noting.

Comment 11

[James Nord]

> I absolutely agree! This bug is potentially very dangerous.

Yup I have lost all my Inbox 3 times  before I worked out what was going on.  

It's second nature for me to select all junk, ctrl-A, del, compact folder...

Comment 12

[Wayne Mery (:wsmwk)]

(In reply to comment #5)
> While the focus is *on* the mailview, the 'A' is grabbed without regard to 
> the Ctrl modifier; and the 'A' is interpreted as the first letter in the 
> "All" MailView.

happens for any ctrl modified key where key matches the first letter of an item in mailview - for example ctrl-s and ctrl-c (save and customize respectively)

Comment 16

[Rod Whiteley]

For details of a hack that seems to fix this, see: http://forums.mozillazine.org/viewtopic.php?p=2971218#2971218

Comment 17

[Wayne Mery (:wsmwk)]

== suite bug 191801

Comment 20

[David Ascher (:davida)]

david, I wonder if the hack described in comment #16 is a worthy proto-patch.

marking as blocking because of the dataloss.

Comment 21

[David :Bienvenu]

phil might have an idea - I don't think we can change the global menulist.xml that way. I tried to quickly add an onkeypress handler to our view menu in mailWindowOverlay.xul but it didn't seem to help.

Comment 22

[Phil Ringnalda (:philor)]

Hmm. Looks to me like (most of) the dataloss aspect was fixed by bug 385374 - the menulist will eat keypresses without respecting modifiers, but will also preventDefault, so ctrl+a will switch you to All, but not select anything.

However, it seems like a menulist bug that ctrl+a is handled by the menulist - HTML selects don't work that way (focus this bug's Product select, and while s takes you to Sumo, ctrl+s wants to save the page), and bug 403210 comment 14 makes it sound like Neil was expecting menulists to not do anything when any of ctrl/alt/meta were used.

Comment 23

[Wayne Mery (:wsmwk)]

compose window, from: field is also afflicted, if you have a name in their that starts with a "W" (of which I have several)

Comment 24

[[e-mail]]

Bug 403210 didn't fix this because modifiers is ignored if you don't specify a key. Oops.

Comment 25

[Wayne Mery (:wsmwk)]

=> fixed by bug 400854.
someone should check the SM bug.