From e10d636ca09e1385b9bf0661062a62cef6227da6 Mon Sep 17 00:00:00 2001 From: Jay Kamat Date: Tue, 9 May 2017 08:08:05 -0700 Subject: [PATCH] Fix a few small issues - Remove an unused warnings - Reverse if statement arguments to simplify logic --- qutebrowser/browser/webengine/webenginetab.py | 9 +++------ 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/qutebrowser/browser/webengine/webenginetab.py b/qutebrowser/browser/webengine/webenginetab.py index e460af049..d4e3feaa2 100644 --- a/qutebrowser/browser/webengine/webenginetab.py +++ b/qutebrowser/browser/webengine/webenginetab.py @@ -17,9 +17,6 @@ # You should have received a copy of the GNU General Public License # along with qutebrowser. If not, see . -# FIXME:qtwebengine remove this once the stubs are gone -# pylint: disable=unused-argument - """Wrapper over a QWebEngineView.""" import functools @@ -281,10 +278,10 @@ class WebEngineCaret(browsertab.AbstractCaret): log.webview.debug("Clicking a searched link via fake key press.") # send a fake enter, clicking the orange selection box - if not tab: - self._tab.key_press(Qt.Key_Enter) - else: + if tab: self._tab.key_press(Qt.Key_Enter, modifier=Qt.ControlModifier) + else: + self._tab.key_press(Qt.Key_Enter) else: # click an existing blue selection