607108 - focus group name field after creating new group

Status: VERIFIED FIXED

(Firefox Graveyard :: Panorama, defect, P4)

Description

[Luis Villa]

When I create a 'new folder' in finder or other file managers, the folder name field is immediately focused so that I can type the new folder's name. 

I have the same expectation when creating a new group, but instead I get search.

Current behavior:
1) click and drag to create new tab group
2) type 'foo' to name the group, hit enter
3) get told there are no search results for foo.

Expected behavior:
1) click and drag to create new tab group
2) type 'foo' to name the group, hit enter
3) the group is now named 'foo'

Would be nice for this to Just Work. :)

Comment 1

[Kevin Hanes]

That certainly seems like the sort of behavior we want! Adding it to the b9 (fit n' finish beta) list so we can discuss it more when we get there.

Comment 2

[Kevin Hanes]

bugspam (moving b9 to b10)

Comment 3

[Kevin Hanes]

bugspam (removing b9)

Comment 4

[Kevin Hanes]

Punting

Comment 5

[Tim Taubert [:ttaubert] (inactive)]

Attachment: patch v1

@Alex: the patch focuses the title input field when using mouse drag to create a new tab group. Shall we also focus the input field when creating a group via dragging a tabItem onto an orphan tab? Without the orphan tab concept we would immediately create a new group when dragging a tab out of its group.

I don't feel like it's the right thing to focus the input field when not explicitly creating a new group but it's up to you :)

Comment 6

[Raymond Lee [:raymondlee]]

Comment on attachment 537601 [details] [diff] [review]
patch v1

+  if (options.focusTitle)
+    this.focusTitle();
+

Nit: Please add a comment about options.focusTitle to the method input variable options.

Comment 7

[Tim Taubert [:ttaubert] (inactive)]

Comment on attachment 537601 [details] [diff] [review]
patch v1

Passed try:

http://tbpl.mozilla.org/?tree=Try&pusher=[e-mail]&rev=1ef959a9e308

Comment 8

[Tim Taubert [:ttaubert] (inactive)]

Attachment: patch v2

Comment 9

[Tim Taubert [:ttaubert] (inactive)]

http://hg.mozilla.org/integration/mozilla-inbound/rev/d8b5c9736831

Comment 10

[Boris Zbarsky [:bzbarsky]]

Backed out due to mochitest-other orange.

Comment 11

[Tim Taubert [:ttaubert] (inactive)]

Attachment: patch v3

Extended patch (and test) to auto-focus the title field when creating a new group by dragging a tab onto an orphan tab (as requested by Limi).

Comment 12

[(no longer active)]

Do we know what was wrong with the original patch which caused this to be backed out?

Comment 13

[Tim Taubert [:ttaubert] (inactive)]

(In reply to comment #12)
> Do we know what was wrong with the original patch which caused this to be
> backed out?

I relanded all other patches except bug 633190 - that was actually causing this intermittent orange. So this patch should be safe.

Comment 14

[Tim Taubert [:ttaubert] (inactive)]

Comment on attachment 538527 [details] [diff] [review]
patch v3

Passed try:

http://tbpl.mozilla.org/?tree=Try&pusher=[e-mail]&rev=3b0da64388be

Comment 15

[Tim Taubert [:ttaubert] (inactive)]

http://hg.mozilla.org/integration/mozilla-inbound/rev/d58376753a93

Comment 16

[Tim Taubert [:ttaubert] (inactive)]

http://hg.mozilla.org/mozilla-central/rev/d58376753a93

Comment 17

[George Carstoiu]

Mozilla/5.0 (X11; Linux i686; rv:7.0a1) Gecko/20110615 Firefox/7.0a1

Verified issue on Ubuntu 11.04 x86, Mac OS X 10.6, Win 7 x86, WinXP using the steps from the Description.

Issue no longer present - changing status to Verified.