Closed
Bug 1279953
Opened 9 years ago
Closed 9 years ago
Ambiguous string usage for "not encrypted page"
Categories
(Firefox :: Page Info Window, defect)
Firefox
Page Info Window
Tracking
()
RESOLVED
FIXED
Firefox 50
Tracking | Status | |
---|---|---|
firefox50 | --- | fixed |
People
(Reporter: nONoNonO, Assigned: nONoNonO)
Details
Attachments
(1 file)
2.78 KB,
patch
|
Dolske
:
review+
|
Details | Diff | Splinter Review |
In the Page Info Window in the security tab, info is shown about the techincal details of the page. According to transvision, there are three states: encrypted, mixed, none. See this URL:
https://transvision.mozfr.org/?recherche=encrypted+page&repo=aurora&sourcelocale=en-US&locale=en-US&search_type=strings
The wording for not encrypted pages is ambiguous. It says:
The page you are viewing is not encrypted.
This should be:
The page you are viewing was not encrypted before being transmitted over the Internet.
Assignee | ||
Comment 1•9 years ago
|
||
Attachment #8762582 -
Flags: review?(dolske)
Comment 2•9 years ago
|
||
Comment on attachment 8762582 [details] [diff] [review]
Fix wording and reference to new entity
r+ as the simplest fix, but...
The "before being transmitted over the Internet" bit of the existing string (dating back to bug 78123) honestly doesn't thrill me. Ideally we should revisit all the wording here -- 15 years later our primary UI is just talking about a "secure connection", and I think it's a little odd to be trying to explain what encryption is (pageInfo_Privacy_Encrypted2).
But this string is buried down in Page Info, and this patch at least makes things more consistent.
Attachment #8762582 -
Flags: review?(dolske) → review+
Assignee | ||
Updated•9 years ago
|
Assignee: nobody → o.e.ekker
Keywords: checkin-needed
Pushed by cbook@mozilla.com:
https://hg.mozilla.org/integration/fx-team/rev/750dc8ef4b2b
Ambiguous string usage for "not encrypted page". r=dolske
Keywords: checkin-needed
Comment 4•9 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 9 years ago
status-firefox50:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 50
You need to log in
before you can comment on or make changes to this bug.
Description
•