rnhmjoj/qutebrowser
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Fix message.confirm_async calls

Browse Source
This commit is contained in:
Florian Bruhin 2016-10-26 19:50:11 +02:00
parent bc4e340323
commit dff20ffe35
2 changed files with 7 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
qutebrowser/browser/urlmarks.py
View File

@ -204,7 +204,8 @@ class QuickmarkManager(UrlMarkManager):
if name in self.marks: if name in self.marks:
message.confirm_async( message.confirm_async(
win_id, "Override existing quickmark?", set_mark, default=True) win_id, title="Override existing quickmark?",
yes_action=set_mark, default=True)
else: else:
set_mark() set_mark()

9
qutebrowser/browser/webkit/webpage.py
View File

@ -137,11 +137,12 @@ class BrowserPage(QWebPage):
# QDesktopServices::openUrl with info.url directly - however it # QDesktopServices::openUrl with info.url directly - however it
# works when we construct a copy of it. # works when we construct a copy of it.
url = QUrl(info.url) url = QUrl(info.url)
msg = "Open external application for {}-link?\nURL: {}".format( scheme = url.scheme()
url.scheme(), url.toDisplayString())
message.confirm_async( message.confirm_async(
self._win_id, msg, self._win_id,
functools.partial(QDesktopServices.openUrl, url)) title="Open external application for {}-link?".format(scheme),
text="URL: {}".format(url.toDisplayString()),
yes_action=functools.partial(QDesktopServices.openUrl, url))
return True return True
elif (info.domain, info.error) in ignored_errors: elif (info.domain, info.error) in ignored_errors:
log.webview.debug("Ignored error on {}: {} (error domain: {}, " log.webview.debug("Ignored error on {}: {} (error domain: {}, "