stonewareslord/treestyletab
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Don't focus to the tab which is going to be removed.

Browse Source 
This fixes #569. In the previous revision, onTabClose method tried to focus to the first child even if it was going to be closed as a member of closing tree. Then TabSelect event was unexpectedly fired, TabWindow.prototype.handleEven (defined in resource:///modules/WindowsPreviewPerTab.jsm) raised an error, and blocked all following event handlers for TabSelect event. As the result, cached tabs list were not cleared and Ctrl-Tab feature was killed.
This commit is contained in:
Piro / YUKI Hiroshi 2013-10-08 02:47:53 +09:00
parent 163df63f04
commit a9c59cc286
1 changed files with 3 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
modules/browser.js
View File

@ -3049,6 +3049,9 @@ TreeStyleTabBrowser.prototype = {
if (nextFocusedTab && toBeClosedTabs.indexOf(nextFocusedTab) > -1) if (nextFocusedTab && toBeClosedTabs.indexOf(nextFocusedTab) > -1)
nextFocusedTab = this.getNextFocusedTab(nextFocusedTab); nextFocusedTab = this.getNextFocusedTab(nextFocusedTab);
if (nextFocusedTab && nextFocusedTab.hasAttribute(this.kREMOVED))
nextFocusedTab = null;
this._reserveCloseRelatedTabs(toBeClosedTabs); this._reserveCloseRelatedTabs(toBeClosedTabs);
this.detachTab(tab, { dontUpdateIndent : true }); this.detachTab(tab, { dontUpdateIndent : true });