Closed Bug 712970 Opened 8 years ago Closed 8 years ago

Show 'tabs from last time' in about:home

Categories

(Firefox for Android :: General, defect, P2)

ARM
Android
defect

Tracking

()

VERIFIED FIXED
Firefox 12
Tracking Status
firefox10 --- verified
firefox11 --- verified
fennec 11+ ---

People

(Reporter: jfu, Assigned: bnicholson)

References

Details

(Keywords: uiwanted, Whiteboard: [MTD][QA+])

Attachments

(3 files, 5 obsolete files)

Web page or screen you were on when you saw the issue: about:home

What you expected: Be able to easily access last open tabs rather than most frequented sites. Since a lot of times I'm on-the-go and need to get off my phone quickly then resume browsing later, having the frequented sites on about:home isn't useful for me. It would be cool to have an option to either set your frequent sites or recent tabs on about:home. 

Has user studies shown otherwise?
Whiteboard: [MTD] → [MTD][DUPEME]
Assignee: nobody → bnicholson
Priority: -- → P2
Summary: option to show recent sites in about:home → Show 'tabs from last time' in about:home
Whiteboard: [MTD][DUPEME] → [MTD]
Tabs should automatically be restored (either by Android staying in memory or via session restore if killed), so I assume this bug is for when the user explicitly chooses to Quit.

What exactly do we want here? A list of tabs on about:home where the user can reopen each individual tab like in XUL Fennec, or do we want a single "Restore Last Session" button like we have on desktop? If the former, should this be in addition to, an alternative to, or a replacement for the frequent sites we currently show on about:home?
I think we want something like XUL Fennec: The list and a "open all"
+1 to what Mark said - however I don't have user data to suggest whether this would be a larger use case and can't determine if it should be replaced. I was envisioning it as an alternative, having an option to either display top sites or the last tabs open.
tracking-fennec: --- → 11+
Attached patch patch (obsolete) — Splinter Review
Attachment #587903 - Flags: review?(blassey.bugs)
Attached patch patch v2 (obsolete) — Splinter Review
Fixed issue where last tabs can't be clicked.
Attachment #587903 - Attachment is obsolete: true
Attachment #587903 - Flags: review?(blassey.bugs)
Attachment #587909 - Flags: review?(blassey.bugs)
Attached patch patch v2 (second attempt) (obsolete) — Splinter Review
I'll even upload the modified patch this time.
Attachment #587909 - Attachment is obsolete: true
Attachment #587909 - Flags: review?(blassey.bugs)
Attachment #587911 - Flags: review?(blassey.bugs)
Attachment #587911 - Flags: review?(blassey.bugs) → review?(mark.finkle)
Comment on attachment 587911 [details] [diff] [review]
patch v2 (second attempt)

>diff --git a/mobile/android/base/AboutHomeContent.java b/mobile/android/base/AboutHomeContent.java

> public class AboutHomeContent extends ScrollView {

>+    public interface JSONReadCallback {
>+        public void callback(String json);
>+    }

Do you use this anywhere? I assume you might have wanted it for a more async loading of JSON. This might be a good idea if the performance of showing the page is regressed.

>diff --git a/mobile/android/base/locales/en-US/android_strings.dtd b/mobile/android/base/locales/en-US/android_strings.dtd

>+<!ENTITY abouthome_addons_browse "Browse all &brandShortName; Add-ons">

"Browse all &brandShortName; add-ons"  <-- lowercase 'a' in add-ons

r+, let's see if this affects performance at all. also, start thinking of how we would write a robotium test for this.
Attachment #587911 - Flags: review?(mark.finkle) → review+
Blocks: 697384
this should be P1 since it blocks us on localization too, right?
So, to confirm:

- this is only when we're not restoring tabs automatically, because the user explicitly quit or we crashed on startup multiple times (the "well, this is embarrassing" case)
- this becomes a section on about:home with a header like the "Add-ons for your Firefox" that appears after the "top sites" section
- the section is a list of tab titles with favicons at left
getting a screenshot of the WIP in here sooner than later would be great, by the way, so we can course correct on the UI.
Keywords: uiwanted
This is removing hard-coded strings in about:home, which we'll need to deal with one way or the other. Tracking 11+ says to me that we want to use this way to fix the hard-coded strings, but that does have an impact on when we actually string freeze, if we're still in uiwanted.
Attached image screenshot (1 last tab) (obsolete) —
Attached image screenshot (2 last tabs) (obsolete) —
(In reply to Madhava Enros [:madhava] from comment #9)
> So, to confirm:
> 
> - this is only when we're not restoring tabs automatically, because the user
> explicitly quit or we crashed on startup multiple times (the "well, this is
> embarrassing" case)
Yes (although we don't yet handle the startup crash loop - that's bug 701092).

> - this becomes a section on about:home with a header like the "Add-ons for
> your Firefox" that appears after the "top sites" section
> - the section is a list of tab titles with favicons at left
Yes and yes (see attached screenshots).
I'd cut the URL line -- tab title should be enough.

I think Ian will have some visual styling comments.
Attached image mockup
Hi Brian, this is looking great. Couple comments on the visual design here:

1. Let's change the url colour to #666666, it's showing up a little too light at the moment.
2. Nudge the url about 2px closer to the page title
3. Apply the same margin to the right of the screen as the left, so the URL doesn't run so close to the edge before getting truncated

Thanks!
Ian

I've attached a mockup for reference
Attached patch patch v3Splinter Review
patch with ui fixes
Attachment #587911 - Attachment is obsolete: true
Attached image screenshot (ui fixes)
Attachment #589288 - Attachment is obsolete: true
Attachment #589289 - Attachment is obsolete: true
Comment on attachment 589313 [details] [diff] [review]
patch v3

r+ previously from mfinkle; new patch contains layout tweaks.
Attachment #589313 - Flags: review+
https://hg.mozilla.org/mozilla-central/rev/2f3527e85663
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 12
Flags: in-litmus?(fennec)
Whiteboard: [MTD] → [MTD][QA^]
Whiteboard: [MTD][QA^] → [MTD][QA+]
Comment on attachment 589313 [details] [diff] [review]
patch v3

[Approval Request Comment] 
adds last tabs to about:home. low risk.
Attachment #589313 - Flags: approval-mozilla-aurora?
Testcase available at:
https://litmus.mozilla.org/manage_testcases.cgi?testcase_id=43045
Flags: in-litmus?(fennec) → in-litmus+
Comment on attachment 589313 [details] [diff] [review]
patch v3

[Triage Comment]
Mobile only - approved for Aurora.
Attachment #589313 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Verified on M-A & M-C
Galaxy Nexus (Android 4.0.3)

20120126053754
http://hg.mozilla.org/releases/mozilla-aurora/rev/9d5ede8cb11a

20120126031113
http://hg.mozilla.org/mozilla-central/rev/402b394b6623
Status: RESOLVED → VERIFIED
OS: Mac OS X → Android
Hardware: x86 → ARM
You need to log in before you can comment on or make changes to this bug.