From 5cf1dce89ed0b44be1f5c5b70e4b642d50246b7f Mon Sep 17 00:00:00 2001 From: Florian Bruhin Date: Sat, 20 Jun 2015 13:06:32 +0200 Subject: [PATCH] Simplify condition and add comment. --- qutebrowser/browser/webview.py | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/qutebrowser/browser/webview.py b/qutebrowser/browser/webview.py index cb6181b7a..7f866e732 100644 --- a/qutebrowser/browser/webview.py +++ b/qutebrowser/browser/webview.py @@ -467,7 +467,13 @@ class WebView(QWebView): # and refocusing it fixes that. self.clearFocus() self.setFocus(Qt.OtherFocusReason) - if len(self.page().selectedText()) == 0: + + # Move the caret to the first element in the viewport if there + # isn't any text which is already selected. + # + # Note: We can't use hasSelection() here, as that's always + # true in caret mode. + if not self.page().selectedText(): self.page().currentFrame().evaluateJavaScript( utils.read_file('javascript/position_caret.js'))