Merge lp:~daker/webbrowser-app/fix.1227055 into lp:webbrowser-app

Proposed by Adnane Belmadiaf
Status: Merged
Approved by: Olivier Tilloy
Approved revision: 327
Merged at revision: 330
Proposed branch: lp:~daker/webbrowser-app/fix.1227055
Merge into: lp:webbrowser-app
Diff against target: 57 lines (+42/-0)
2 files modified
src/app/Browser.qml (+1/-0)
src/app/CertificateVerificationDialog.qml (+41/-0)
To merge this branch: bzr merge lp:~daker/webbrowser-app/fix.1227055
Reviewer Review Type Date Requested Status
PS Jenkins bot continuous-integration Approve
Olivier Tilloy Approve
Review via email: mp+186890@code.launchpad.net

Commit message

Added certificateVerificationDialog component

To post a comment you must log in.
Revision history for this message
Olivier Tilloy (osomon) wrote :

42 + // TRANSLATORS: %1 refers to the URL of the current website

One tiny nitpick: %1 refers to the hostname, not the full URL of the website. Can you please update the comment?

Regarding the translation template updates, since there are other pending MRs that add new strings, I suggest we wait and update the template once all those changes have been merged.

review: Needs Fixing
Revision history for this message
Olivier Tilloy (osomon) wrote :

Looks great, thanks!

review: Approve
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
lp:~daker/webbrowser-app/fix.1227055 updated
327. By Adnane Belmadiaf

Merged trunk
Fixed merge conflicts

Revision history for this message
Adnane Belmadiaf (daker) wrote :

Can you please reapprove this.

Revision history for this message
PS Jenkins bot (ps-jenkins) :
review: Approve (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/app/Browser.qml'
2--- src/app/Browser.qml 2013-09-23 14:48:21 +0000
3+++ src/app/Browser.qml 2013-09-23 20:22:39 +0000
4@@ -346,6 +346,7 @@
5 devicePixelRatio: browser.qtwebkitdpr
6
7 experimental.preferences.developerExtrasEnabled: browser.developerExtrasEnabled
8+ experimental.certificateVerificationDialog: CertificateVerificationDialog { }
9 experimental.authenticationDialog: AuthenticationDialog {}
10 experimental.proxyAuthenticationDialog: ProxyAuthenticationDialog {}
11 experimental.alertDialog: AlertDialog { }
12
13=== added file 'src/app/CertificateVerificationDialog.qml'
14--- src/app/CertificateVerificationDialog.qml 1970-01-01 00:00:00 +0000
15+++ src/app/CertificateVerificationDialog.qml 2013-09-23 20:22:39 +0000
16@@ -0,0 +1,41 @@
17+/*
18+ * Copyright 2013 Canonical Ltd.
19+ *
20+ * This file is part of webbrowser-app.
21+ *
22+ * webbrowser-app is free software; you can redistribute it and/or modify
23+ * it under the terms of the GNU General Public License as published by
24+ * the Free Software Foundation; version 3.
25+ *
26+ * webbrowser-app is distributed in the hope that it will be useful,
27+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
28+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
29+ * GNU General Public License for more details.
30+ *
31+ * You should have received a copy of the GNU General Public License
32+ * along with this program. If not, see <http://www.gnu.org/licenses/>.
33+ */
34+
35+import QtQuick 2.0
36+import Ubuntu.Components 0.1
37+import Ubuntu.Components.Popups 0.1 as Popups
38+
39+Popups.Dialog {
40+ title: i18n.tr("This connection is untrusted")
41+ // TRANSLATORS: %1 refers to the hostname
42+ text: i18n.tr("You are trying to securely reach %1, but the security certificate of this website is not trusted.").arg(model.hostname)
43+
44+ Button {
45+ text: i18n.tr("Proceed anyway")
46+ color: "red"
47+ onClicked: model.accept()
48+ }
49+
50+ Button {
51+ text: i18n.tr("Back to safety")
52+ color: "green"
53+ onClicked: model.reject()
54+ }
55+
56+ Component.onCompleted: show()
57+}

Subscribers

People subscribed via source and target branches

to status/vote changes: