Ian Gilman [:iangilman] Reporter
	Description • 14 years ago

If you close the last tab in a window from within that tab, the window should close as it does. If, however, you close the last tab from within the TabCandy UI, you should still be sitting there in TabCandy. Currently, closing the last tab in the TabCandy UI does close the whole window.

	Raymond Lee [:raymondlee] Assignee
	Comment 1 • 14 years ago

Closing last tab from within TabCandy would mean no tab exists in that window. I see a problem that if there is no tab in a window, we can't show anything (not even a "blank" tab) when user quits TabCandy.

	Ian Gilman [:iangilman] Reporter
	Comment 2 • 14 years ago

What do you mean by quitting TabCandy? Do you mean zooming back in to a tab? It's true that shouldn't be allowed if there are no tabs to zoom back into.

	Aza Raskin [:aza]
	Comment 3 • 14 years ago

+1 to Ian's question.

Raymond:Once you close the last tab inside of the TabCandy interface, you cannot hide the TabCandy interface in that window (or in your terminology "quit TabCandy") because there are no tabs to zoom to.

	Raymond Lee [:raymondlee] Assignee
	Comment 4 • 14 years ago

I am able to show the close the last tab inside the TabCandy interface and the widow doesn't close.  However, some errors are thrown when I press the + button in the TabCandy interface.  The errors are related to  the content.document are not found since no tab exists.  The new tab is added to the right of the "+" button on the tabstrip instead of the left.  I am going to look into those issues.

	Raymond Lee [:raymondlee] Assignee
	Comment 5 • 14 years ago

I have created a patch for this bug but there is still an issue.  Removing the last tab from a window and then adding some new tabs, all of the tabs show up on the right of the "new tab" button.  The patch uses the standard methods: tabbrowser.removeTab(tab) to remove the last tab, and the use tabbrowser.addTab(url) to add new tabs.  

There was a hack fixed the same issue in the "tabbrowser-tabs" binding, see
http://mxr.mozilla.org/mozilla-central/source/browser/base/content/tabbrowser.xml#2492

However, it arises again when we remove last tab from the binding and add new tabs again.

@Aza/Ian: Is it possible to get some advices from someone who is familiar with binding code?

	Ian Gilman [:iangilman] Reporter
	Comment 6 • 14 years ago

Patch applied in: 

http://hg.mozilla.org/users/edward.lee_engineering.uiuc.edu/tabcandy-central/rev/3654c66d044e

Mardak, any ideas who to talk to about the remaining issue?

	Aza Raskin [:aza]
	Comment 7 • 14 years ago

Adding Paul to see if he has any insights into the problems described in comment 5.

	(no longer active)
	Comment 8 • 14 years ago

Mass moving all Tab Candy bugs from Mozilla Labs to Firefox::Tab Candy.  Filter the bugmail spam with "tabcandymassmove".

	alex_mayorga
	Comment 10 • 14 years ago

From bug 576427
Closing the only set actually close(s) the tab(s) AND Firefox.
Clearly a bug as I had "Warn me when closing multiple tabs" option
enabled but I wasn't warned at all.
Worst first use experience ever as there's not an obvious "mouseable" way to get out. Being vocal as this one really annoyed me when it bite me the first time =(

	[not reading bugmail]
	Comment 11 • 14 years ago

It would help if TabCandy was a secondary window, not replacing the main window.

	abittner
	Comment 12 • 14 years ago

it would help if closing (X for example on right upper corner of window/border or similar function in all of the different operating systems) the tabcandy outer window would result in simply going back where you came from, from the tabcandy-less view of the firefox window containing multiple tabs, and closing the grouped representation of the tabs inside the tabcandy view (the X functionality there) would result in a question if you were sure and would only continue there, etc....

when the resoning is that the tabcandy view takes over the main firefox window with/without multiple tabs open, and closing this window with the outer X of the window, then it should behave the same way as the original window.

behaviour of my firefox is, that it asks me if i was sure if i have more than a single tab open in that window, and it simply closes when i only have a single tab/area opened there.

thanks.
https://bugzilla.mozilla.org/show_bug.cgi?id=588355

	Aza Raskin [:aza]
	Comment 14 • 14 years ago

Raymond, can you confirm that this is still a bug (see comment #5).

	Raymond Lee [:raymondlee] Assignee
	Comment 15 • 14 years ago

(In reply to comment #14)
> Raymond, can you confirm that this is still a bug (see comment #5).

I recall that someone mentioned we are not allowed to remove the last tab in the browser and keep the browser open.  Therefore, the patch in comment #6 was backed out.  In other words, if you close all tabs in tab view UI, the browser would get closed.

	Michael Yoshitaka Erlewine [:mitcho]
	Comment 16 • 14 years ago

(In reply to comment #15)
> (In reply to comment #14)
> > Raymond, can you confirm that this is still a bug (see comment #5).
> 
> I recall that someone mentioned we are not allowed to remove the last tab in
> the browser and keep the browser open.

Raymond, out of curiosity, was that a UX consideration or something necessary for us to pass tests?

	Raymond Lee [:raymondlee] Assignee
	Comment 17 • 14 years ago

(In reply to comment #16)
> Raymond, out of curiosity, was that a UX consideration or something necessary
> for us to pass tests?

It wasn't a UX consideration or something to do with tests.  It was about the architecture which supports browser window without a tab but I guess there must be something we can do to support that.

	Raymond Lee [:raymondlee] Assignee
	Comment 18 • 14 years ago

(In reply to comment #17)
> It wasn't a UX consideration or something to do with tests.  It was about the
> architecture which supports browser window without a tab but I guess there must
> be something we can do to support that.

Oops, I mean the architecture which doesn't support browser window without any tabs.

	Aza Raskin [:aza]
	Comment 19 • 14 years ago

Interesting. I could see how Firefox wouldn't support the case of a tab-less window. Sounds like something we'll have to fiddle with to get working right. Charge forth with the fiddling!

	Ian Gilman [:iangilman] Reporter
	Comment 20 • 14 years ago

This needs to be sorted out now that we have Panorama across all windows.  We're creating a UX where it's possible to load any tab into any window, but behind the scenes, each tab is hosted on a specific window at any one time.  This means it's possible that going into a group in one window will cause another window to close (if that other window had only the tabs for the group in question).  

We could possibly hack around it by moving tabs from other group or orphan tab to that window to keep it open, but even that won't work in some cases.  

We need to make it possible for a window to not contain any tabs.  If that occurs, that window needs to enter the tabcandy UI.

	Aza Raskin [:aza]
	Comment 21 • 14 years ago

Agreed. A Firefox window needs to be able to exist without any tabs (but only if it is in Tab Candy mode).

	Raymond Lee [:raymondlee] Assignee
	Comment 22 • 14 years ago

Who is the best person we can get some advices from about window without any tabs?

	Aza Raskin [:aza]
	Comment 23 • 14 years ago

Gavin, perhaps?

	Raymond Lee [:raymondlee] Assignee
	Comment 24 • 14 years ago

Gavin: please see comment #20 and comment #21

	Dão Gottwald [:dao]
	Comment 25 • 14 years ago

It's not clear what would break when a window has zero tabs, but it might be a lot. Not just within tabbrowser, but everything watching tabs could potentially break, add-ons as well as stock features like Aero Peek. TabCandy should probably make sure there's at least one group in each window; if there aren't enough groups left, it should in fact let the window close.

	Ian Gilman [:iangilman] Reporter
	Comment 26 • 14 years ago

(In reply to comment #25)
> It's not clear what would break when a window has zero tabs, but it might be a
> lot. Not just within tabbrowser, but everything watching tabs could potentially
> break, add-ons as well as stock features like Aero Peek. TabCandy should
> probably make sure there's at least one group in each window; if there aren't
> enough groups left, it should in fact let the window close.

Aza, what would you say from a UX perspective?  Seems weird that windows would arbitrarily close (from the user's perspective).  If we want this change, we should make it sooner than later to shake out the potential problems Dao mentions.

	Aza Raskin [:aza]
	Comment 27 • 14 years ago

Having a window close unexpectedly when clicking on a tab in another window's Panorama is confusing and scary. For that reason alone we should allow windows to have zero tabs.

A second and just as compelling reason is this: Imagine you have two windows A and B with groups a and b, respectively. You want to perform the simple operation of switching the to look at group b in window A, and group a in window B. You'd like to be able to go to Panorama in both windows and just click the group you want. If windows are not allowed to have zero tasks, however, this will fail in a strange and unexpected way. This is what will happen:

* In window A, you open Panorama.
* In window B, you open Panorama. (optional)
* In window A, you click on group b. Window B now closes.

Lastly, whether a window closes or not is dependent on the internal state of which window actually houses a tab. The user has no idea (and shouldn't have any idea) which tab is currently actually housed in which window. That is an implementation detail that shouldn't be exposed. But it is exposed when a window closes because no tabs are housed there, even if only temporarily.

	Aza Raskin [:aza]
	Comment 28 • 14 years ago

To give a sense of how wrong this feels, watch the following movie (http://people.mozilla.org/~araskin/panbugs/Closing.mov)

	Dietrich Ayala (:dietrich)
	Comment 30 • 14 years ago

Blocking+, for confusing and scary.

However, I don't understand why you're requiring zero tabs? Why not just use a new-window's default? (usually the user's home page, or about:blank). That shouldn't be overly surprising, since it's what a user sees anytime they open a new window.

	Tan Wei Lin Jason [:Ryuji]
	Comment 31 • 14 years ago

That would like forcing user to their home page or a blank page whenever they close all tabs in Tab Candy UI, which may not be what they want.

	Ian Gilman [:iangilman] Reporter
	Comment 32 • 14 years ago

The recent discussion in this bug has focused around the behavior in "all windows" (bug 578512). Now that that feature is getting bumped, we should refocus this discussion on what we're shipping with FF4 (or decide to bump this as well). 

Given how Panorama works now, closing the last tab and having the window go away is indeed jarring, but not nearly as much as shown in Aza's video; you'll never be in a situation where you close a tab in one window and have another window go away. 

What can still happen is you close all the tabs in a window (while in the Panorama UI), and the window goes away. Perhaps not what you expected, but pretty easy to understand what happened, and certainly no data loss. Not ideal perhaps, but livable.

	Aza Raskin [:aza]
	Comment 33 • 14 years ago

Ideally: Closing the last tab inside of Tab Candy shouldn't close the window. To implement this (WARNING: might be a terrible idea) we can upon closing the last tab open a hidden collapsed tab whose entire purpose is to give the window a tab so as to not close. The user would never see the hidden tab so this is entirely an implementation details.

Not-as-ideally: We let the window close when you close the last tab in Tab Candy. I would not mark this blocking.

	alex_mayorga
	Comment 34 • 14 years ago

Who knew the whole close Firefox when closing last tab bug 456405 would come and haunt us two years down the road.
I personally hated it back in the day, but somehow learned to live with it =)
Maybe bug 455852 can help here.

	Ginn Chen
	Comment 35 • 14 years ago

At least we should warn user, it is going to quit Firefox.
If I quit firefox with multiple tabs, I got a warning.
But if I close it in "Group you tabs ...", I got nothing.

	Ian Gilman [:iangilman] Reporter
	Comment 36 • 14 years ago

(In reply to comment #35)
> At least we should warn user, it is going to quit Firefox.
> If I quit firefox with multiple tabs, I got a warning.
> But if I close it in "Group you tabs ...", I got nothing.

That reminds me of another thing... with bug 587341 (undo close group), if you close the last group in Tab Candy (leaving no more tabs), your window will continue to exist for the next 15 seconds (while the "undo" button remains), but then the window will close (if you haven't made any new tabs in that time). Could be a little disconcerting.

	Aza Raskin [:aza]
	Comment 37 • 14 years ago

This is admittedly weird. That said, I don't think it is too weird. My preference, however, would be for for the last group's undo button to stay around indefinitely until the browser window is manually closed.

	Ian Gilman [:iangilman] Reporter
	Comment 38 • 14 years ago

(In reply to comment #37)
> This is admittedly weird. That said, I don't think it is too weird. My
> preference, however, would be for for the last group's undo button to stay
> around indefinitely until the browser window is manually closed.

That could be done! File a bug?

	Aza Raskin [:aza]
	Comment 39 • 14 years ago

File-up bug followed: http://bugzilla.mozilla.org/show_bug.cgi?id=595521

	Ian Gilman [:iangilman] Reporter
	Comment 40 • 14 years ago

This bug has been marked as "future" (post-ff4.0) but it still blocked our b8 milestone bug; removing blocking.

	TGOS
	Comment 41 • 14 years ago

> What can still happen is you close all the tabs in a window
> (while in the Panorama UI), and the window goes away. Perhaps
> not what you expected, but pretty easy to understand what
> happened, and certainly no data loss.

Right now there is a data loss: All my groups are gone! If I created 20 groups, took the hassle to name them all and layout them on my Panorama view (position and size) and then I close the last tab of the only group that still had one, it's all gone. All 20 groups are lost, their names and their layout.

If the window was still there, I could still create a new tab in one of the groups, surf around from there, spawning many new tabs, going back to Panorama view, moving some of the tabs to the already existing groups (with names and layout) and so on.

This is the main reason I'm currently not using Panorama at all. It's just too much hassle to name and/or layout groups, if I know those will be lost sooner or later anyway.