Closed Bug 49232 Opened 24 years ago Closed 24 years ago

Should only be one tab from addressing field to subject line

Categories

(MailNews Core :: Composition, defect, P2)

Product:
MailNews Core
Component:
Composition
Type:
defect

Tracking

(Not tracked)

Status:
VERIFIED FIXED

People

(Reporter: lchiang, Assigned: hewitt)

References

Details

(Whiteboard: [nsbeta3-][PDTP2][rtm+ need info])

Attachments

(1 file)

patch with fixes as described above
5.78 KB, text/plain
Details 
Should only be one tab from addressing field to subject line

This is taken from 10932.  Bug 10932 is getting long in its description.  Per 
mail triage, we will make tabbing work from addressing field to subject line.  
Here are the scenarios tested:

Win32 2000-08-16-10-m18

A) HTML and plain text compose with one addressing line
1.  New message
2.  Type an address in To: field
3.  Tab
4.  Tab
5.  Tab - cursor appears in Subject field.  Type in a subject
6.  Tab - cursor appears in the Body field.  Type something in the body
Problem:  Takes three tabs to get from the recipient field to the body.

B) HTML and plain text compose with two addressing lines
1.  New message
2.  Type an address in To: field
3.  Enter.  Second addressing field appears. Type an address.
4.  Tab
5.  Tab
6.  Tab - cursor appears in Subject field.  Type in a subject
7.  Tab - cursor appears in the Body field.  Type something in the body
Problem:  Takes three tabs to get from the recipient field to the body.
Keywords: nsbeta3
QA Contact: lchiang → laurel
JF- can you find a new owner for this?  does it belong to toolkit group?
Keywords: mailtrack
Peter, we think this is a toolkit thing.  Can you find an owner for it?  We 
would like to see this make beta3, let me know if you can't do that.
Assignee: ducarroz → trudelle
->saari.  Isn't it up to apps authors to determine what elements the tab key
cycles through?
Assignee: trudelle → saari
*** Bug 48971 has been marked as a duplicate of this bug. ***
Yeah, tab order is working now, so they should be able to do this...

Saari, let me know what's the syntax to define tabs order and reassign this bug 
to me. Thanks
reassigning to ducarroz
Assignee: saari → ducarroz
rteassign to varada...
Assignee: ducarroz → varada
Chris, please add the syntax information to this bug.  Thanks!
removing mailtrack since assigned to mailnews eng.

+, P2 per mail triage
Keywords: mailtrack → mail2
Priority: P3 → P2
Whiteboard: [nsbeta3+]
Target Milestone: --- → M18
adding self to cc list

accepting
Status: NEW → ASSIGNED
PDT agrees P2. Current behavior seems to be that you can't tab into the body, no
matter how many tabs you hit. You have to set focus with the mouse.
Whiteboard: [nsbeta3+] → [nsbeta3+][PDTP2]
bug 52014 addresses not being able to get to the message body by tabbing 
anymore.
Adding UE1 keyword to this child bug.
Keywords: UE1
Marking Fixed.
The tab from addressing field to subject line to message box works.
Status: ASSIGNED → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
Using sept20 commercial build:
It takes Two tabs to get from addressing area to subject line, but tabbing does
work... are we calling that OK or are the requirements One tab?
It was working create last week for few day but it has regress again since!!! I 
think we should reopen it...
yeah, still takes > 1 tab, build from 9/20/00, 8am
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
If this doesn't make b3, we'd like to consider it for rtm.
Keywords: rtm
It's working again for me. At least on Mac. Varada, can you verify it is working 
on all platform and then close this bug again :-)
It still takes two tabs on Windows for me with this morning's build.
this didn't work with a build from 9/21/00. two tabs required. fwiw.

Clarification: It's working fine except from the firt recipient field!

similare problem occurs when pressing enter from the first recipient: the focus 
doesn't go to the next field! I spoke with saari yesterday about this xplatform 
problem and he is looking at it. 
it takes me two tabs from the 2nd recipient also.
rtm+, this is one of our UE1 top UE issues.
Whiteboard: [nsbeta3+][PDTP2] → [nsbeta3+][PDTP2][rtm+]
Not holding PR3 for this, so marking nsbeta3-
Whiteboard: [nsbeta3+][PDTP2][rtm+] → [nsbeta3-][PDTP2][rtm+]
varada, you should reassign this bug to saari.
don't forget to add the keyword mailtriage before reassigning this bug.

Saari, did you look at this remaining problem (press return from the first 
field don't set the focus to the next one)?
Reassigning to Saari (focus issue) and adding mailtrack
Assignee: varada → saari
Status: REOPENED → NEW
Keywords: mailtrack
no, I haven't got a chance to look at this yet
Status: NEW → ASSIGNED
Heh, Murphy's Law. This works on my Mac build from today, 10/2

It still doesn't work on my Windows build from today

On linux, mac, NT PR3 builds: still takes a couple tabs.  Until there's a
tangible fix, no more comments from me.

Really? I just tried this on my Win2K build with a pull from about 2pm and it
works okay.

It *might* be some of my code that I've been mucking around with (windows
specific focus fixes)

I'll pull those changes out and see what happens...

I figured out why this works for me; I run the classic skin. This bug is
specific to Modern. My suggested fix? Use a better skin.

Marking needinfo.  Cc'ing Hangas as this may be skin related.
Whiteboard: [nsbeta3-][PDTP2][rtm+] → [nsbeta3-][PDTP2][rtm+ needinfo]
Whiteboard: [nsbeta3-][PDTP2][rtm+ needinfo] → [nsbeta3-][PDTP2][rtm+ need info]
Assignee: saari → hangas
Status: ASSIGNED → NEW
Modern only -> hangas

Sending to Joe, he is pretty sure he can fix this in the messagecompose css file.
Assignee: hangas → hewitt
I've posted a patch for this to bug 54750.  I'll post the same patch here, but 
note that it contains other fixes for the address tree which are related to 
this one.  Basically just had to add a -moz-user-focus: ignore to 
#addressingWidgetTree, since it was stealing focus after you tab out of the 
textfield.
Status: NEW → ASSIGNED
r=ducarroz. I applied the patch and tested it in Classic & Modern. appart two 
little detail Joe already fixed, it works fine.
sr=mscott

Fixed on trunk and branch.
Status: ASSIGNED → RESOLVED
Closed: 24 years ago → 24 years ago
Resolution: --- → FIXED
OK with plain text and html compose window, modern and classic skin.
OK with:
2000-10-10-10 mn6 commercial branch build linux rh6.0
2000-10-10-08 mn6 commercial branch build NT 4.0
2000-10-10-08 mn6 commercial branch build mac OS 9.0

adding vtrunk keyword
Keywords: vtrunk
Verified Fixed on trunk builds  Tested plain and HTML compose, classic and
modern themes.
linux 101708 RedHat 6.2
win32 101704 NT 4
mac 101704 Mac OS9
Setting bug to Verified and removing vtrunk keyword
Status: RESOLVED → VERIFIED
Keywords: vtrunk
Keywords: rtm, UE1 → nsrtm
Product: MailNews → Core
Product: Core → MailNews Core
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: