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 | ||||
Related bugs: |
|
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 certificateVeri
To post a comment you must log in.
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.