[Tablet] "Undo close tab" thumbnail stays around too long

VERIFIED FIXED in Firefox 9

Status

Fennec Graveyard
General
VERIFIED FIXED
6 years ago
4 years ago

People

(Reporter: mbrubeck, Assigned: mbrubeck)

Tracking

(Blocks: 1 bug)

Firefox 9
Firefox 9
All
Android

Details

Attachments

(1 attachment)

(Assignee)

Description

6 years ago
In the phone UI, the "undo close tab" thumbnail goes away when the sidebar is dismissed.  Since you can't dismiss the sidebar in the tablet UI, the thumbnail just sticks around indefinitely.  We should make it disappear at some point.

Here are some situations where we could choose to make it disappear:

* User opens a new tab
* User switches tabs
* User navigates to a new page in an open tab
* User takes some explicit action to clear the "undo" thumbnail
* Ten minutes (or whatever) has elapsed
(Assignee)

Comment 1

6 years ago
Created attachment 560000 [details] [diff] [review]
patch

This removes the closed tab thumbnail when:
* User switches tabs
* User opens a new tab
* User navigates to a new page using the location bar

I also considered closing it whenever the user navigates from one page to another (by clicking a link, for example) but since location changes so frequently and are not always user-initiated, so I thought this might cause the closed tab thumbnail to disappear too soon or too unexpectedly.

Any comments from the UX team?
Attachment #560000 - Flags: review?(mark.finkle)
(Assignee)

Comment 2

6 years ago
(In reply to Matt Brubeck (:mbrubeck) from comment #1)
> Any comments from the UX team?

Madhava says yes, we should try out this patch.
Status: NEW → ASSIGNED
Keywords: uiwanted
OS: Linux → Android
Hardware: x86 → All
Comment on attachment 560000 [details] [diff] [review]
patch


>   selectTab: function selectTab(aTab) {
>     AwesomeScreen.activePanel = null;
>     Browser.selectedTab = aTab;
>+    Elements.tabList.removeClosedTab();
>   },

Maybe this would be better in _tabSelect, since any code could set Browser.selectedTab to something.

r+, but I think we should consider the above
Attachment #560000 - Flags: review?(mark.finkle) → review+
(Assignee)

Comment 4

6 years ago
https://hg.mozilla.org/mozilla-central/rev/0a6d1368cd97

(In reply to Mark Finkle (:mfinkle) from comment #3)
> Maybe this would be better in _tabSelect, since any code could set
> Browser.selectedTab to something.

_tabSelect gets called too often - for example, when you close a tab it causes another tab to get selected, but we don't want to remove the closed tab thumbnail *right* after you close it.
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 9
Verified fixed on:
Mozilla/5.0 (Android;Linux armv7l;rv:9.0a1)Gecko/20110926
Firefox/9.0a1 Fennec/9.0a1
Device: Acer ICONIA A500
OS: Android 3.1
Status: RESOLVED → VERIFIED
tracking-fennec: ? → ---
You need to log in before you can comment on or make changes to this bug.