[browser] limit the amount of open tabs restored when browser starts

Bug #1376433 reported by Bill Filler
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ubuntu UX
Fix Released
Medium
Unassigned
webbrowser-app
Fix Released
High
Olivier Tilloy
webbrowser-app (Ubuntu)
Fix Released
Undecided
Unassigned

Bug Description

I just noticed when launching the browser I had a huge list of open tabs (like 30). We should limit this to the 10 most recently visited tabs. Otherwise the list is bound to continue to grow and it's not really apparent to the user unless they visit the open tabs page. Even though we are not loading them in memory I'm sure there is still overhead with creating them.

Related branches

Bill Filler (bfiller)
Changed in webbrowser-app:
importance: Undecided → Critical
assignee: nobody → Olivier Tilloy (osomon)
tags: added: rtm14 touch-2014-10-09
Revision history for this message
Olivier Tilloy (osomon) wrote :

WebViews are instantiated on demand, when a tab is first activated. So when you start the browser and it restores 30 previously open tabs, it actually instantiates 30 lightweight tabComponent objects which are made up of just a FocusScope, hidden by default.

So I would expect the overhead to be really minimal, if not negligible.

It might still make sense to limit the number of tabs restored (given our data model in which tabs are always ordered by recency, this should be trivial to implement).

Adding an ubuntu-ux task to get design to comment on the idea.

Revision history for this message
Giorgio Venturi (giorgio-venturi-deactivatedaccount) wrote :

Hi Bill,

Thank you for raising the good point.
We actually designed a feature to limit the amount of displayed tabs to 10, with an action to show 10 more, then 10 more up to 40 tabs.
https://docs.google.com/a/canonical.com/presentation/d/1Qrd4Flfs3EH-fI79IfrYgLdAx2nce-L7ve8NKLCX324/edit#slide=id.g2bb3fbfe6_03
We could implement this feature so every time you restart the browser, only the first 10 are displayed.

I'd rather have that, than removing the tabs altogether from the data model.

I hope it makes sense.

Changed in ubuntu-ux:
status: New → Opinion
assignee: nobody → Giorgio Venturi (giorgio-venturi)
importance: Undecided → Low
Revision history for this message
Olivier Tilloy (osomon) wrote :

Lowering importance to 'High' as this doesn’t impair significantly the user experience, nor is it a crash.

Changed in webbrowser-app:
importance: Critical → High
status: New → Triaged
Revision history for this message
Bill Filler (bfiller) wrote : Re: [Bug 1376433] Re: limit the amount of open tabs restored when browser starts

Until we can implement what Giorgio suggests, let's just limit the list to 10. In the future we can add the buttons for showing 10 more and 10 more after that as specified.

> On Oct 3, 2014, at 4:58 AM, Olivier Tilloy <email address hidden> wrote:
>
> Lowering importance to 'High' as this doesn’t impair significantly the
> user experience, nor is it a crash.
>
> ** Changed in: webbrowser-app
> Importance: Critical => High
>
> ** Changed in: webbrowser-app
> Status: New => Triaged
>
> --
> You received this bug notification because you are subscribed to the bug
> report.
> https://bugs.launchpad.net/bugs/1376433
>
> Title:
> limit the amount of open tabs restored when browser starts
>
> To manage notifications about this bug go to:
> https://bugs.launchpad.net/ubuntu-ux/+bug/1376433/+subscriptions

Revision history for this message
Joe Odukoya (jodukoya) wrote : Re: limit the amount of open tabs restored when browser starts

+1

Olivier Tilloy (osomon)
Changed in webbrowser-app:
status: Triaged → In Progress
Revision history for this message
Giorgio Venturi (giorgio-venturi-deactivatedaccount) wrote :

Shall I file the feature above as a separate bug?

Revision history for this message
Olivier Tilloy (osomon) wrote :

Yes, please do. Thanks!

Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package webbrowser-app - 0.23+14.10.20141008-0ubuntu1

---------------
webbrowser-app (0.23+14.10.20141008-0ubuntu1) utopic; urgency=low

  [ CI bot ]
  * Resync trunk

  [ Alexandre Abreu ]
  * Properly setup host support when devtools inspector mode used (LP:
    #1376783)
  * Add pdf mime type support to content hub mapping, (LP: #1374611)

  [ Olivier Tilloy ]
  * Remove the temporary workaround for bug #1328839 as fullscreen
    switching is now fully functional in qtubuntu. (LP: #1328839)
  * Remove an unused import that triggered a (harmless) warning. (LP:
    #1377885)
  * Update application screenshot. (LP: #1377974)
  * Restore only the n most recent tabs at startup, to limit the
    overhead of instantiating too many tab objects. (LP: #1376433)
 -- Ubuntu daily release <email address hidden> Wed, 08 Oct 2014 09:23:52 +0000

Changed in webbrowser-app (Ubuntu):
status: New → Fix Released
Olivier Tilloy (osomon)
Changed in webbrowser-app:
status: In Progress → Fix Released
Revision history for this message
Giorgio Venturi (giorgio-venturi-deactivatedaccount) wrote :
Changed in ubuntu-ux:
status: Opinion → Fix Committed
importance: Low → Medium
summary: - limit the amount of open tabs restored when browser starts
+ [browser] limit the amount of open tabs restored when browser starts
Changed in ubuntu-ux:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.