Closed
Bug 1140777
Opened 10 years ago
Closed 10 years ago
[UX] Review and revisit integration of mixed content blocking in control center
Categories
(Firefox :: General, defect)
Tracking
()
RESOLVED
FIXED
Iteration:
40.1 - 13 Apr
People
(Reporter: phlsa, Assigned: agrigas)
References
Details
(Whiteboard: [ux])
Attachments
(1 file, 5 obsolete files)
(deleted),
image/png
|
Details |
The remaining piece of structural work for control center is to fix the integration of the mixed content blocker. Currently, mixed content blocking and tracking protection share the shield icon. Going forward, that is probably not the best long term solution.
This bug covers the treatment of mixed content blocking in both the identity block and the control center doorhanger.
Wireframe of doorhanger:
https://bugzilla.mozilla.org/attachment.cgi?id=8574348
Wireframe of identity block:
bug 1125226
Flags: firefox-backlog+
Updated•10 years ago
|
Flags: qe-verify-
Updated•10 years ago
|
Assignee: nobody → agrigas
Status: NEW → ASSIGNED
Iteration: --- → 39.2 - 23 Mar
Points: --- → 8
Assignee | ||
Comment 1•10 years ago
|
||
Draft 2 of control center for mixed content
Attachment #8581637 -
Flags: ui-review?(philipp)
Assignee | ||
Comment 2•10 years ago
|
||
new version based on feedback from Tanvi
Attachment #8581637 -
Attachment is obsolete: true
Attachment #8581637 -
Flags: ui-review?(philipp)
Attachment #8581840 -
Flags: ui-review?(philipp)
Assignee | ||
Comment 3•10 years ago
|
||
revision
Attachment #8581840 -
Attachment is obsolete: true
Attachment #8581840 -
Flags: ui-review?(philipp)
Attachment #8581891 -
Flags: ui-review?(philipp)
Assignee | ||
Comment 4•10 years ago
|
||
Attachment #8581891 -
Attachment is obsolete: true
Attachment #8581891 -
Flags: ui-review?(philipp)
Attachment #8581908 -
Flags: ui-review?(philipp)
Assignee | ||
Comment 5•10 years ago
|
||
Also see updated wiki for all states here: https://mana.mozilla.org/wiki/display/FIREFOX/Polaris+Tracking+Protection+UX
Updated•10 years ago
|
Iteration: 39.2 - 23 Mar → 39.3 - 30 Mar
Assignee | ||
Comment 6•10 years ago
|
||
Attachment #8581908 -
Attachment is obsolete: true
Attachment #8581908 -
Flags: ui-review?(philipp)
Attachment #8585507 -
Flags: feedback?(tanvi)
Updated•10 years ago
|
Iteration: 39.3 - 30 Mar → 40.1 - 13 Apr
Assignee | ||
Comment 7•10 years ago
|
||
Closing this out and any changes needed will happen in visual design.
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Comment 8•10 years ago
|
||
Comment on attachment 8585507 [details]
v6.png
Hi Aislinn,
What will the UI look like for non-EV + mixed active content blocked?
For EV + mixed active content blocked, we show a green lock with a grey yield sign.
For non-EV we have to show a grey lock. But I don't think we can put a grey yield sign on it. And the yellow yield sign means something else.
Assignee | ||
Comment 9•10 years ago
|
||
Added state for non-ev and mixed active blocked
Attachment #8585507 -
Attachment is obsolete: true
Attachment #8585507 -
Flags: feedback?(tanvi)
Comment 10•10 years ago
|
||
Going to summarize the latest design https://bug1140777.bugzilla.mozilla.org/attachment.cgi?id=8586254
Overall policy:
When something is a bit off on a secure page, we indicate to the user that there is an issue by adding a yield sign to the lock. The yield sign can be different colors based on it's severity. When a user disables protection, they lose the lock entirely.
Breakdown:
When a page is fully secure with no attempt to load any type of mixed content we show a green lock for EV pages and a grey lock for non-EV pages.
State 1: Mixed active content is blocked on an EV page -
green lock + grey yield sign (replaces current grey shield + green lock)
State 2: Mixed active content is blocked on an non-EV page -
grey lock + white yield sign (replaces current grey shield + grey lock)
State 3: Mixed passive content loaded (EV or non-EV)
grey lock + yellow yield sign (replaces current grey yield sign)
State 4: Mixed passive content loaded AND mixed active content blocked (EV or non-EV)
grey lock + yellow yield sign (replaces current grey shield + grey yield sign)
State 5: Mixed active content loaded (EV or non-EV):
yellow yield sign (no change to current behavior)
Comment 11•10 years ago
|
||
There seems to be a subtle grey background shade for state 4. Is that an intentional signal, or just an artifact of editing images? Either way it's fine.
Assignee | ||
Comment 12•10 years ago
|
||
(In reply to Daniel Veditz [:dveditz] from comment #11)
> There seems to be a subtle grey background shade for state 4. Is that an
> intentional signal, or just an artifact of editing images? Either way it's
> fine.
Unintentional - mock error will be refined in visual design. Thanks!
Comment 13•10 years ago
|
||
(In reply to Tanvi Vyas [:tanvi] from comment #10)
> When a page is fully secure with no attempt to load any type of mixed
> content we show a green lock for EV pages and a grey lock for non-EV pages.
I think we should continue to keep the green motif for both the lock and the site owner's name, to indicate that the two are bound together, as we do now. The site owner's name should be at least as visually prominent as the domain name, as it's harder to spoof/do a lookalike, and much easier for humans to reason about.
> State 1: Mixed active content is blocked on an EV page -
> green lock + grey yield sign (replaces current grey shield + green lock)
>
> State 2: Mixed active content is blocked on an non-EV page -
> grey lock + white yield sign (replaces current grey shield + grey lock)
I think the white sign (which, BTW, indicated "Yield" only in the US :-) is rather hard to see on the light grey background.
Why is there a need for a grey sign and a white sign? What is the user supposed to understand from the difference?
> State 3: Mixed passive content loaded (EV or non-EV)
> grey lock + yellow yield sign (replaces current grey yield sign)
>
> State 4: Mixed passive content loaded AND mixed active content blocked (EV
> or non-EV)
> grey lock + yellow yield sign (replaces current grey shield + grey yield
> sign)
>
> State 5: Mixed active content loaded (EV or non-EV):
> yellow yield sign (no change to current behavior)
The text here "you have disabled protection from insecure content" seems not very helpful. Also, the second shot says "Secure connection" but then the error seems to suggest that it's not secure. Is changing that in scope for this work? I'd prefer something like "You have requested that Firefox load insecure content on this site", and "Insecure connection".
Gerv
Assignee | ||
Comment 14•10 years ago
|
||
(In reply to Gervase Markham [:gerv] from comment #13)
> (In reply to Tanvi Vyas [:tanvi] from comment #10)
> > When a page is fully secure with no attempt to load any type of mixed
> > content we show a green lock for EV pages and a grey lock for non-EV pages.
>
> I think we should continue to keep the green motif for both the lock and the
> site owner's name, to indicate that the two are bound together, as we do
> now. The site owner's name should be at least as visually prominent as the
> domain name, as it's harder to spoof/do a lookalike, and much easier for
> humans to reason about.
>
> > State 1: Mixed active content is blocked on an EV page -
> > green lock + grey yield sign (replaces current grey shield + green lock)
> >
> > State 2: Mixed active content is blocked on an non-EV page -
> > grey lock + white yield sign (replaces current grey shield + grey lock)
>
> I think the white sign (which, BTW, indicated "Yield" only in the US :-) is
> rather hard to see on the light grey background.
>
> Why is there a need for a grey sign and a white sign? What is the user
> supposed to understand from the difference?
>
> > State 3: Mixed passive content loaded (EV or non-EV)
> > grey lock + yellow yield sign (replaces current grey yield sign)
> >
> > State 4: Mixed passive content loaded AND mixed active content blocked (EV
> > or non-EV)
> > grey lock + yellow yield sign (replaces current grey shield + grey yield
> > sign)
> >
> > State 5: Mixed active content loaded (EV or non-EV):
> > yellow yield sign (no change to current behavior)
>
> The text here "you have disabled protection from insecure content" seems not
> very helpful. Also, the second shot says "Secure connection" but then the
> error seems to suggest that it's not secure. Is changing that in scope for
> this work? I'd prefer something like "You have requested that Firefox load
> insecure content on this site", and "Insecure connection".
>
> Gerv
Hi Gerv,
Thanks for your comments. We were back and forth on text color and I was going to let the visual design make recommendations as that is the next step for this ticket. This was for the interaction design. I will convey your suggestions. To clarify, the gray and white sign are options for the same state. We'll aim to get the most readable treatment in visual design. The secure connection text when mixed active is not blocked should be insecure, that is a mock error I've since corrected.
Comment 15•10 years ago
|
||
(In reply to agrigas from comment #7)
> Closing this out and any changes needed will happen in visual design.
Aislinn, do you have a bug number for the visual design? Do you know what kind of time frame we have for this?
Comment 16•10 years ago
|
||
(In reply to Gervase Markham [:gerv] from comment #13)
> (In reply to Tanvi Vyas [:tanvi] from comment #10)
> > State 2: Mixed active content is blocked on an non-EV page -
> > grey lock + white yield sign (replaces current grey shield + grey lock)
>
> I think the white sign (which, BTW, indicated "Yield" only in the US :-) is
> rather hard to see on the light grey background.
The US Yield sign isn't that different from the rest of the world, and it has never been a triangle with the point facing up. https://en.wikipedia.org/wiki/Yield_sign
Triangle with the point up is a common shape for warning signs around the world. (Warning road signs are more commonly diamond shaped in the US, but the triangle is used for warnings in other contexts.)
Assignee | ||
Comment 17•10 years ago
|
||
(In reply to Tanvi Vyas [:tanvi] from comment #15)
> (In reply to agrigas from comment #7)
> > Closing this out and any changes needed will happen in visual design.
>
> Aislinn, do you have a bug number for the visual design? Do you know what
> kind of time frame we have for this?
Tanvi - this is the visual design bug: https://bugzilla.mozilla.org/show_bug.cgi?id=1140783
Stephen Horlander is working on it. For release timing, you would have to ask Javaun. I have no updates there.
Updated•10 years ago
|
You need to log in
before you can comment on or make changes to this bug.
Description
•