Allow closing tabs in the Ctrl+Tab panel with Ctrl+W

VERIFIED FIXED in Firefox 3.1a2

Status

()

--
enhancement
VERIFIED FIXED
11 years ago
10 years ago

People

(Reporter: dao, Assigned: dao)

Tracking

Trunk
Firefox 3.1a2
Points:
---
Dependency tree / graph
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(1 attachment, 3 obsolete attachments)

(Assignee)

Description

11 years ago
From <http://en.design-noir.de/mozilla/ctrl-tab/?com#c:71>:

"Great job!
But I miss a feature. When ctrl+tabbing I would like to be able to close the tab by pressing ctrl+W.
Didn’t get it? This is what i would like:
Hold ctrl
Tap tab(to open your wonderful extension/choose tab)
Tap W(to close selected tab)

In my opinion it’s much easier than switching tab before you can close it."
(Assignee)

Updated

11 years ago
Summary: Allow closing a tab in the Ctrl-Tab panel → Allow closing tabs in the Ctrl+Tab panel with Ctrl+W
(Assignee)

Comment 1

11 years ago
Created attachment 329710 [details] [diff] [review]
patch
Assignee: nobody → dao
Status: NEW → ASSIGNED
(Assignee)

Updated

11 years ago
Flags: wanted-firefox3.1?
Keywords: uiwanted
(Assignee)

Comment 2

11 years ago
Created attachment 329741 [details] [diff] [review]
better patch
Attachment #329710 - Attachment is obsolete: true
(Assignee)

Comment 3

11 years ago
Comment on attachment 329741 [details] [diff] [review]
better patch

This causes trouble with browser.ctrlTab.previewsCount > 3. Probably a good idea to drop that pref anyway.
Attachment #329741 - Attachment is obsolete: true
(Assignee)

Updated

11 years ago
Attachment #329799 - Flags: ui-review?(beltzner)
(Assignee)

Comment 5

11 years ago
This can be tested with the latest version of the extension.
(Assignee)

Updated

11 years ago
Blocks: 445573
(Assignee)

Updated

11 years ago
Attachment #329799 - Flags: review?(gavin.sharp)
(Assignee)

Updated

11 years ago
Blocks: 446065
(Assignee)

Comment 6

11 years ago
Created attachment 330358 [details] [diff] [review]
v3, updated and merged
Attachment #329799 - Attachment is obsolete: true
Attachment #330358 - Flags: ui-review?(beltzner)
Attachment #330358 - Flags: review?(gavin.sharp)
Attachment #329799 - Flags: ui-review?(beltzner)
Attachment #329799 - Flags: review?(gavin.sharp)
(Assignee)

Updated

11 years ago
Flags: wanted-firefox3.1?
As described in bug 445495 it would improve the user experience if you don't hard-code the number of shown previews to 3. Having a long list of tabs will make it really hard to find the right one. Further using 5 previews would be a good initial value.
(Assignee)

Comment 8

11 years ago
See bug 445495 comment 6.
Comment on attachment 330358 [details] [diff] [review]
v3, updated and merged

Yeah, this makes sense.

We should probably also look at adding support for using the left and right arrows as a way of moving back and forth while the preview window is open; is there already a bug on that?
Attachment #330358 - Flags: ui-review?(beltzner) → ui-review+
(Assignee)

Comment 10

10 years ago
(In reply to comment #9)
> We should probably also look at adding support for using the left and right
> arrows as a way of moving back and forth while the preview window is open; is
> there already a bug on that?

No, I'll file one.
Keywords: uiwanted
Comment on attachment 330358 [details] [diff] [review]
v3, updated and merged

It would be nice if this code had a few more comments in general. Some of the "magic numbers" particularly are a bit opaque especially now that some of the code assumes a constant visibleCount==3.
Attachment #330358 - Flags: review?(gavin.sharp) → review+
(Assignee)

Comment 12

10 years ago
I don't think any code that I touch here assumes visibleCount==3. (Otherwise, I'd like to fix that.) The magic numbers are due to the fact that this.selected is always the second-visible preview, even if you set browser.ctrlTab.previewsCount > 3 without this patch applied. I'll add some comments in this regard.
(Assignee)

Comment 13

10 years ago
http://hg.mozilla.org/index.cgi/mozilla-central/rev/d5743c68c91a
Status: ASSIGNED → RESOLVED
Last Resolved: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 3.1a2
Depends on: 450049
The fix regressed bug 450049 at least on OS X.
Verified with:

Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:1.9.1a2pre) Gecko/20080812022114 Minefield/3.1a2pre ID:20080812022114

Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.1a2pre) Gecko/20080812045727 Minefield/3.1a2pre ID:20080812045727
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.