Closed
Bug 747087
Opened 13 years ago
Closed 13 years ago
Make the https:// black (to match the domain color) in the https non-ev case
Categories
(Firefox :: Address Bar, defect)
Firefox
Address Bar
Tracking
()
RESOLVED
WONTFIX
People
(Reporter: curtisk, Assigned: jaws)
References
()
Details
(Whiteboard: [SecReview Action Item][blocks FF15])
Attachments
(2 files)
(deleted),
patch
|
Details | Diff | Splinter Review | |
(deleted),
patch
|
Details | Diff | Splinter Review |
No description provided.
Updated•13 years ago
|
Component: Security Assurance → Location Bar
Product: mozilla.org → Firefox
QA Contact: security-assurance → location.bar
Version: other → Trunk
Comment 1•13 years ago
|
||
I'm worried that making "https" black will make it harder to pick out the domain name. I'd prefer making "https" blue or green.
Comment 2•13 years ago
|
||
How about we just leave it as is? The fact that we show https:// at all while hiding http:// is already an emphasis, and more importantly, there's the lock icon.
Comment 3•13 years ago
|
||
(In reply to Jesse Ruderman from comment #1)
> I'd prefer making "https" blue or green.
+1 for blue, green is for HTTPS+EV according to bug 747085.
(In reply to Dão Gottwald [:dao] from comment #2)
> more importantly, there's the lock icon.
beltzner has raised some, very valid IMHO, concerns on "the return of the padlock" at https://groups.google.com/d/topic/mozilla.dev.apps.firefox/ODX1PJutsLM/discussion you might be interested in joining the discussion there.
Comment 4•13 years ago
|
||
Before the new design, the site identity button was blue for the non-ev case (https://support.mozilla.org/en-US/kb/Site%20Identity%20Button#w_blue-basic-identity-information), however probably it's better to let it be black (otherwise less aware people wouldn't understand the difference between blue and green!)
Assignee | ||
Comment 5•13 years ago
|
||
The security team requested that we darken the https:// portion to make it easier to see that the connection is over SSL during the security review.
This patch makes domain-verified connections have a black https://. Tests to follow.
Attachment #621132 -
Flags: review?(dao)
Assignee | ||
Comment 6•13 years ago
|
||
Attachment #621133 -
Flags: review?(dao)
Comment 7•13 years ago
|
||
(In reply to Dão Gottwald [:dao] from comment #2)
> How about we just leave it as is? The fact that we show https:// at all
> while hiding http:// is already an emphasis, and more importantly, there's
> the lock icon.
I agree with Dão.
We are going to make the lock darker also. I don't think it needs more emphasis than that.
Assignee | ||
Comment 8•13 years ago
|
||
Comment on attachment 621132 [details] [diff] [review]
Patch for bug
Clearing review since it looks like we don't want to do this anymore.
Attachment #621132 -
Flags: review?(dao)
Assignee | ||
Updated•13 years ago
|
Attachment #621133 -
Flags: review?(dao)
Assignee | ||
Comment 9•13 years ago
|
||
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•