Closed Bug 1603526 Opened 5 years ago Closed 5 years ago

Focus on reply should be in message body, is in To field instead

Categories

(Thunderbird :: Message Compose Window, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
Thunderbird 73.0

People

(Reporter: jik, Assigned: aleca)

References

(Regression)

Details

(Keywords: regression)

Attachments

(2 files, 1 obsolete file)

This is a regression caused by the changes for bug 440377 that landed earlier today.

When I reply to a message my keyboard focus should start in the message body, but instead it's in the To field.

Status: NEW → UNCONFIRMED
Ever confirmed: false
Flags: needinfo?(alessandro)

Thanks for reporting this, it's extremely helpful having as much feedback as possible on this early implementation that just landed.
Confirmed. I'll take care of it.

Flags: needinfo?(alessandro)
Assignee: nobody → alessandro
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Attached patch 1603526-msgbody-focus.patch (obsolete) (deleted) — Splinter Review

This should fix it.

Attachment #9115572 - Flags: review?(mkmelin+mozilla)
Comment on attachment 9115572 [details] [diff] [review] 1603526-msgbody-focus.patch Review of attachment 9115572 [details] [diff] [review]: ----------------------------------------------------------------- Seems to work, thx! r=mkmelin ::: mail/components/compose/content/MsgComposeCommands.js @@ +4790,5 @@ > > function AdjustFocus() { > let element = document.getElementById("toAddrInput"); > + if ( > + element.value == "" && I guess you don't need this anymore?
Attachment #9115572 - Flags: review?(mkmelin+mozilla) → review+
Attached patch 1603526-msgbody-focus.patch (deleted) — Splinter Review

You're right, that value will always be empty.
I tweaked it a bit to not use closest(), since the container we need has its own ID.

Attachment #9115572 - Attachment is obsolete: true
Attachment #9115601 - Flags: review+

Try run?

(In reply to Magnus Melin [:mkmelin] from comment #6)

Try run?

Agh, sorry, I completely forgot about it:
https://treeherder.mozilla.org/#/jobs?repo=try-comm-central&revision=6026e1f4c9b7f54e34ef01e5998909bd8c351382

It hasn't finished yet, let's see if something unusual pops up.

Pushed by mkmelin@iki.fi:
https://hg.mozilla.org/comm-central/rev/08af3c83117b
Fix focus on message body when compose window is opened by a reply action. r=mkmelin

Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 73.0

(In reply to Pulsebot from comment #8)

Pushed by mkmelin@iki.fi:
https://hg.mozilla.org/comm-central/rev/08af3c83117b
Fix focus on message body when compose window is opened by a reply action. r=mkmelin

This fixed the problem in Mail, but does NOT fix the same Problem when you answer a newsgroup-posting!
And BTW: the To-Field is superfluous in Newsgroups and should not be shown.

(In reply to ruediger.lahl from comment #10)

(In reply to Pulsebot from comment #8)

Pushed by mkmelin@iki.fi:
https://hg.mozilla.org/comm-central/rev/08af3c83117b
Fix focus on message body when compose window is opened by a reply action. r=mkmelin

This fixed the problem in Mail, but does NOT fix the same Problem when you answer a newsgroup-posting!
And BTW: the To-Field is superfluous in Newsgroups and should not be shown.

Still happening in Thunderbird 73.0b1 also with newsgroup followups.

Status: RESOLVED → REOPENED
Resolution: FIXED → ---

(In reply to WaltS48 [:walts48] from comment #11)

(In reply to ruediger.lahl from comment #10)

(In reply to Pulsebot from comment #8)

Pushed by mkmelin@iki.fi:
https://hg.mozilla.org/comm-central/rev/08af3c83117b
Fix focus on message body when compose window is opened by a reply action. r=mkmelin

This fixed the problem in Mail, but does NOT fix the same Problem when you answer a newsgroup-posting!
And BTW: the To-Field is superfluous in Newsgroups and should not be shown.

Still happening in Thunderbird 73.0b1 also with newsgroup followups.

And of course in 74.01a Dailys.

Attached image removeTo.png (deleted) —

Image showing the To: field in a newsgroup composition window.

We're taking care of this in bug 1607526.
I'll take a look at why the focus doesn't work with newsgroups.

Thanks! I'll close this one again.

Status: REOPENED → RESOLVED
Closed: 5 years ago5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: