Bug 12813 - REGRESSION: In Gmail tabbing through compose has regressed unproductively
Summary: REGRESSION: In Gmail tabbing through compose has regressed unproductively
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Evangelism (show other bugs)
Version: 420+
Hardware: Mac OS X 10.4
: P1 Normal
Assignee: Nobody
URL: http://mail.google.com/mail/
Keywords: GoogleBug, InRadar, Regression
Depends on:
Blocks:
 
Reported: 2007-02-19 09:50 PST by Stephen Harbage
Modified: 2007-04-11 18:40 PDT (History)
5 users (show)

See Also:


Attachments
Reduction (132 bytes, text/html)
2007-03-02 15:17 PST, Beth Dakin
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Stephen Harbage 2007-02-19 09:50:42 PST
Summery:
In Gmail>Compose, tabbing from To, to Subject, and then logically and most helpfully to body of the message doesn't work as it did in 419.3

Steps to reproduce:
Go to Gmail>Compose
Press tab (moves from To: to Subject)
Press tab (moves from Subject to Send button)

Whereas in 419.3 moves to main box to type in message
Comment 1 Adam Roben (:aroben) 2007-02-19 11:57:23 PST
Stephen, I'm unable to reproduce this on a ToT debug build (r19707). Can you provide some more details about your configuration? Do you have the "Press Tab to highlight each item on a webpage" Safari preference turned on? What about universal keyboard access?
Comment 2 Andrew Wellington 2007-02-20 02:45:11 PST
I can reproduce this.

Mac OS X 10.4.8 PowerPC
Safari 2.0.4 (419.3)
WebKit r19727

Safari is not set to tab to every item on the page.
Keyboard & Mouse System Preferences are set to tab to all controls

This does not occur with WebKit 418.9.1, however Gmail serves it the basic compose field, whereas the ToT WebKit receives a rich text editor.

The compose field on ToT is an iframe element, not a form textara element as in the release Safari's page.
Comment 3 Stephen Harbage 2007-02-20 07:05:28 PST
Hi Adam,
(In reply to comment #1)
> Stephen, I'm unable to reproduce this on a ToT debug build (r19707). Can you
> provide some more details about your configuration? Do you have the "Press Tab
> to highlight each item on a webpage" Safari preference turned on?
 No I don't have this turned on
> What about universal keyboard access?
If you mean full keyboard access [in Keyboard and Mouse Prefs], then yes it is on for 'all controls'.

Interestingly, if I switch this preference to 'text boxes and lists only' tabbing through Gmail>Compose Mail goes from to:, subject: and then google search at the top in Gmail (this appears to be another knock on effect from Safari's bad handling of the rich text box and tabbing - notice it works fine in plain text and also in FFX 2.0.0.1 which ever way the full keyboard access is set)

Andrew - nice one for spotting about the rich text and plain text editor. So basically, I found the same:
- In rich text tabbing doesn't work properly
- Switch it to plain text it works fine

I guess technically then this shouldn't be a regression issue as for Safari rich text is a new feature for composing in Gmail. Although it kind of is a regression since it breaks the way Safari properly handles the default way users will access Gmail>Compose Mail and then tabbing through it, which in WebKit 418.9.1 works fine
Comment 4 Maciej Stachowiak 2007-02-27 19:35:07 PST
<rdar://problem/5028157>
Comment 5 Beth Dakin 2007-03-02 15:17:22 PST
Created attachment 13455 [details]
Reduction

I debugged this some and was able to come up with this reduction. The problem is that the Subject field and the buttons at the bottom have their tabIndex set to 1. The iframe (the message body) does not have a tabIndex attribute set, so it defaults to 0. 

Adam (and Geoff?) recently did a bunch of work with our tab order stuff, and the current implementation matches the HTML4 spec. This works in Firefox because Firefox has a lot of tab order bugs as compared with the spec. 

This could be fixed very easily from the GMail side by adding a tabIndex=1 attribute to the iframe. In fact, we may need to fix this bug that way. I'm not sure we will be willing to change our behavior since it appears to match the spec. (Though I will verify that it matches the spec first, of course.)
Comment 6 Beth Dakin 2007-03-02 15:48:35 PST
Here is the part of the spec that addresses tab order:

http://www.w3.org/TR/html401/interact/forms.html#h-17.11.1

Our behavior is clearly correct. The one place that we diverge from the spec is that we support tabindex on more elements that the spec lists, including iframe, meaning that the GMail fix I suggested above does actually work. It looks like all browsers support tabindex on more elements than the spec indicates.

Anyway, this seems like an evangelism issue after all.
Comment 7 David Kilzer (:ddkilzer) 2007-03-02 16:05:30 PST
This bug needs some Google-evangelism-love per Comment #5 and Comment #6.

Comment 8 Aaron Whyte 2007-03-14 18:10:05 PDT
Alrighty then, I'll take a look.  Thanks for the detailed report.
Comment 9 Aaron Whyte 2007-03-15 11:23:56 PDT
Fix checked in on the main branch.  It'll be a week or two before it hits production.
Comment 10 Stephen Harbage 2007-03-15 15:26:45 PDT
Thanks. I assume that will apply to both the US and UK and other versions of Gmail?
Comment 11 Darin Adler 2007-04-11 02:28:21 PDT
Antti tried this on 3/22 and said it was indeed fixed on the Google side.
Comment 12 Stephen Harbage 2007-04-11 16:59:55 PDT
Only on the US site. Please can Google update it for all other languages on Gmail as well?

(In reply to comment #11)
> Antti tried this on 3/22 and said it was indeed fixed on the Google side.
> 
Comment 13 Matt Lilek 2007-04-11 18:40:17 PDT
(In reply to comment #12)
> Only on the US site. Please can Google update it for all other languages on
> Gmail as well?
> 

Reclosing this bug since there is nothing more we can do on our end - it's entirely up to Google to decide what updates they release and when they choose to do so.