Closed Bug 450743 Opened 16 years ago Closed 16 years ago

Limit the number of Ctrl+Tab previews in MRU order, display the rest in tab bar order

Categories

(Firefox :: Tabbed Browser, enhancement)

enhancement
Not set
normal

Tracking

()

VERIFIED FIXED
Firefox 3.1b1

People

(Reporter: dao, Assigned: dao)

References

Details

Attachments

(1 file, 1 obsolete file)

Attached patch patch (obsolete) — Splinter Review
People can't remember the full chronology of their tabs, so the order seemingly becomes random at some point, where it makes sense to fall back to the tab bar order. Windows does something similar for Alt+Tab.
Attached patch patch v2Splinter Review
more optimized sorting
Attachment #333932 - Attachment is obsolete: true
Comment on attachment 334507 [details] [diff] [review] patch v2 In the upcoming ctrl-tab overhaul, I'm going to add support for browser.ctrlTab.recentlyUsedLimit=-1 as a means to disable the limit.
Attachment #334507 - Flags: review?(gavin.sharp)
Attachment #334507 - Flags: review?(gavin.sharp) → review+
Keywords: checkin-needed
pushed aeeef43eb774 I made the pref default to 7.
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 3.1b1
Verified with Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:1.9.1b1pre) Gecko/20080927020752 Minefield/3.1b1pre ID:20080927020752 Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.1b1pre) Gecko/20080927033433 Minefield/3.1b1pre ID:20080927033433 Marcia, do we already have a Litmus test therefor?
Status: RESOLVED → VERIFIED
I still think any MRU scheme is going to be too confusing and disorienting (see bug 445499 Comment #53 for explanation). I hope this bug will be made INVALID (and the patch removed) and bug 445499 will be reopened and fixed. https://bugzilla.mozilla.org/show_bug.cgi?id=445499#c53
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: