118050 - Caret (cursor) is positioned after the signature if you change the identity (From: address)

Status: RESOLVED FIXED

(MailNews Core :: Composition, defect)

Description

[llanero]

From Bugzilla Helper:
User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:0.9.7) Gecko/20011221
BuildID:    2001122108

If you change the from address before writing the message, the signature will
also change (this is correct), but when you hit tab to start typing the message
at the top, the cursor will show at the bottom, while it should appear at the
top, to start typing the message.

Reproducible: Always
Steps to Reproduce:
1. Create a new message
2. Change the from adress
3. Go to the message body: the cursor will start at the bottom (after the
signature), but it should go to the top, before the signature.

Comment 1

[Boris Zbarsky [:bzbarsky]]

composition

Comment 2

[Sheela Ravindran]

Confirming this behavior from 2002-01-07-06 win98 build.
Tabbing to compose area puts the cursor below the signature.  It should put the
cursor at the beginning.  
Might have a known bug on this. 

Comment 3

[http://tinymailto.com/oliversl]

I Mozilla 1.2 Build ID: 2002111512,
the cursor move to the end of the line before the last line.
For example, if this is your signature:
ABCDE
FGHIJ
KLMNO
The cursor will be just after the caracter 'J' and before the caracter 'K'

Comment 4

[Mike Cowperthwaite]

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

Comment 5

[Mike Cowperthwaite]

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

Comment 6

[Mike Cowperthwaite]

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

Comment 7

[Mike Cowperthwaite]

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

Comment 12

[Luke Taylor]

Exactly the same problem here.

Mozilla/5.0 (Windows; U; Windows NT 5.1; en-GB; rv:1.9.2.13) Gecko/20101207 Thunderbird/3.1.7

Same problem on ubuntu too (installed via package manager & download direct from website).

When composing a new message, using tab or enter to move to the next field would usually put the cursor above the signature ready to type the message body.

However when you change identity/from address, using tab or enter will put the cursor beyond the signature, rather than above the "--" signature delimiter.

Duplicate bug reports: 590178, 604986

Comment 13

[domi]

test with TB 13.1 under windows and linux

the problem is linked with the identity switch while replying to a mail

with the standard identity the signature is OK

in my case (reply above citation  and signature under reply above citation)

after replying i get

* first line in normal (black) body format
* second line in signature (grey) format  sometimes with "--"
* third line with signature text

case 1)
if i switch identity for the reply before typing anything
i LOST the normal (black) first line
* the first line is now in grey signature format
* the second line is the signature

case 2)
i write something before switching identity
the old signature (that was after my reply) is erased
and the new signature is put top of the body BEFORE my reply

?? link with bug 604986 703086 118050 736551

Comment 16

[Michal Bruncko]

Very annoying bug. I found this problem a long time before but now I decided to report it and I just found this bugreport...
clicking on vote button.. but I am definitely not sure if this somehow helps with progress...

Comment 17

[vinnie]

I just discovered this issue today in version 91.2.1 when switching sending accounts and tabbing down into the composition box. The cursor is positioned below the new sending account's signature instead of above it.

Comment 18

[Hartmut Welpmann [:welpy-cw]]

This bug got my attention when calling the MailExtension function compose.setComposeDetails() in the Add-On I maintain moved the cursor as well, see corresponding issue. Interestingly, code preserving the caret position when switching already exists in MsgComposeCommands.js, but is only executed in listener NotifyComposeBodyReady() before calling LoadIdentity(). The later function, however, is called directly by SetComposeDetails(), so moving the code saving and restoring the caret position into LoadIdentity() itself seems to fix this bug.

Comment 19

[Hartmut Welpmann [:welpy-cw]]

Attachment: Bug 118050 - Move code saving caret position into LoadIdentity(). r=mkmelin,darktrojan

Comment 20

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/14acef43351f
Move code saving caret position into LoadIdentity(). r=mkmelin