Closed
Bug 712970
Opened 13 years ago
Closed 13 years ago
Show 'tabs from last time' in about:home
Categories
(Firefox for Android Graveyard :: General, defect, P2)
Tracking
(firefox10 verified, firefox11 verified, fennec11+)
VERIFIED
FIXED
Firefox 12
People
(Reporter: jfu, Assigned: bnicholson)
References
Details
(Keywords: uiwanted, Whiteboard: [MTD][QA+])
Attachments
(3 files, 5 obsolete files)
(deleted),
image/png
|
Details | |
(deleted),
patch
|
bnicholson
:
review+
akeybl
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
(deleted),
image/png
|
Details |
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?
Updated•13 years ago
|
Whiteboard: [MTD] → [MTD][DUPEME]
Updated•13 years ago
|
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]
Assignee | ||
Comment 1•13 years ago
|
||
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?
Comment 2•13 years ago
|
||
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.
Updated•13 years ago
|
tracking-fennec: --- → 11+
Assignee | ||
Comment 4•13 years ago
|
||
Attachment #587903 -
Flags: review?(blassey.bugs)
Assignee | ||
Comment 5•13 years ago
|
||
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)
Assignee | ||
Comment 6•13 years ago
|
||
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)
Updated•13 years ago
|
Attachment #587911 -
Flags: review?(blassey.bugs) → review?(mark.finkle)
Comment 7•13 years ago
|
||
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+
Comment 8•13 years ago
|
||
this should be P1 since it blocks us on localization too, right?
Comment 9•13 years ago
|
||
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
Comment 10•13 years ago
|
||
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.
Comment 11•13 years ago
|
||
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.
Assignee | ||
Comment 12•13 years ago
|
||
Assignee | ||
Comment 13•13 years ago
|
||
Assignee | ||
Comment 14•13 years ago
|
||
(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).
Comment 15•13 years ago
|
||
I'd cut the URL line -- tab title should be enough.
I think Ian will have some visual styling comments.
Comment 16•13 years ago
|
||
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
Assignee | ||
Comment 17•13 years ago
|
||
patch with ui fixes
Attachment #587911 -
Attachment is obsolete: true
Assignee | ||
Comment 18•13 years ago
|
||
Attachment #589288 -
Attachment is obsolete: true
Attachment #589289 -
Attachment is obsolete: true
Assignee | ||
Comment 19•13 years ago
|
||
Comment on attachment 589313 [details] [diff] [review]
patch v3
r+ previously from mfinkle; new patch contains layout tweaks.
Attachment #589313 -
Flags: review+
Comment 20•13 years ago
|
||
Updated•13 years ago
|
status-firefox10:
--- → affected
status-firefox11:
--- → affected
Comment 21•13 years ago
|
||
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 12
Updated•13 years ago
|
Flags: in-litmus?(fennec)
Whiteboard: [MTD] → [MTD][QA^]
Updated•13 years ago
|
Whiteboard: [MTD][QA^] → [MTD][QA+]
Assignee | ||
Comment 22•13 years ago
|
||
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?
Comment 23•13 years ago
|
||
Testcase available at:
https://litmus.mozilla.org/manage_testcases.cgi?testcase_id=43045
Flags: in-litmus?(fennec) → in-litmus+
Comment 24•13 years ago
|
||
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+
Comment 25•13 years ago
|
||
Comment 26•13 years ago
|
||
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
Updated•4 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•