diff --git a/qutebrowser/completion/models/miscmodels.py b/qutebrowser/completion/models/miscmodels.py index 657a73c8c..62267d51c 100644 --- a/qutebrowser/completion/models/miscmodels.py +++ b/qutebrowser/completion/models/miscmodels.py @@ -206,10 +206,10 @@ class TabCompletionModel(base.BaseCompletionModel): tabbed_browser = objreg.get('tabbed-browser', scope='window', window=win_id) if not tabbed_browser.shutting_down: - window_count = window_count + 1 + window_count += 1 if window_count < self.rowCount(): - self.removeRows(window_count, self.rowCount()-window_count) + self.removeRows(window_count, self.rowCount() - window_count) for i, win_id in enumerate(objreg.window_registry): tabbed_browser = objreg.get('tabbed-browser', scope='window', @@ -223,15 +223,15 @@ class TabCompletionModel(base.BaseCompletionModel): c.setData("{}".format(win_id), Qt.DisplayRole) if tabbed_browser.count() < c.rowCount(): c.removeRows(tabbed_browser.count(), - c.rowCount()-tabbed_browser.count()) + c.rowCount() - tabbed_browser.count()) for idx in range(tabbed_browser.count()): tab = tabbed_browser.widget(idx) if idx >= c.rowCount(): - self.new_item(c, "{}/{}".format(win_id, idx+1), + self.new_item(c, "{}/{}".format(win_id, idx + 1), tab.url().toDisplayString(), tabbed_browser.page_title(idx)) else: - c.child(idx, 0).setData("{}/{}".format(win_id, idx+1), + c.child(idx, 0).setData("{}/{}".format(win_id, idx + 1), Qt.DisplayRole) c.child(idx, 1).setData(tab.url().toDisplayString(), Qt.DisplayRole) @@ -250,9 +250,7 @@ class TabCompletionModel(base.BaseCompletionModel): qtutils.ensure_valid(category) index = category.child(index.row(), self.IDX_COLUMN) win_id, tab_index = index.data().split('/') - win_id = int(win_id) - tab_index = int(tab_index) tabbed_browser = objreg.get('tabbed-browser', scope='window', - window=win_id) - tabbed_browser.on_tab_close_requested(tab_index-1) + window=int(win_id)) + tabbed_browser.on_tab_close_requested(int(tab_index) - 1)