Closed
Bug 766985
Opened 11 years ago
Closed 11 years ago
Don't show the warning triangle icon for sites with mixed content
Categories
(Firefox :: Theme, defect)
Firefox
Theme
Tracking
()
RESOLVED
FIXED
Firefox 16
People
(Reporter: shorlander, Assigned: shorlander)
References
Details
Attachments
(3 files)
8.37 KB,
patch
|
dao
:
review+
lsblakk
:
approval-mozilla-beta+
|
Details | Diff | Splinter Review |
8.48 KB,
patch
|
dao
:
review+
Gavin
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
7.78 KB,
patch
|
dao
:
review+
|
Details | Diff | Splinter Review |
The warning icon is too alarming and frequent, turning up on any https site that displays http images. Until we can differentiate between different kinds of mixed content we should just show the default globe instead of the lock.
Comment 1•11 years ago
|
||
Agreed - let's get to the right UI in the next version rather than doing this now.
Updated•11 years ago
|
Component: Location Bar → Theme
QA Contact: location.bar → theme
Assignee | ||
Comment 2•11 years ago
|
||
[Approval Request Comment] Bug caused by bug 747090 and bug 755429 User impact if declined: Warning icon desensitization, unnecessary anxiety and confusion about meaning Risk to taking this patch: Low risk CSS removal and image deletion String or UUID changes made by this patch: None
Attachment #635866 -
Flags: review?(jaws)
Attachment #635866 -
Flags: approval-mozilla-beta?
Assignee | ||
Comment 3•11 years ago
|
||
Attachment #635867 -
Flags: review?(jaws)
Assignee | ||
Comment 4•11 years ago
|
||
Attachment #635869 -
Flags: review?(jaws)
Updated•11 years ago
|
Attachment #635869 -
Flags: review?(jaws) → review+
Updated•11 years ago
|
Attachment #635867 -
Flags: review?(jaws) → review+
Updated•11 years ago
|
Attachment #635866 -
Flags: review?(jaws) → review+
Assignee | ||
Comment 5•11 years ago
|
||
Comment on attachment 635867 [details] [diff] [review] Remove Mixed State Icon - Aurora [Approval Request Comment] Bug caused by bug 747090 and bug 755429 User impact if declined: Warning icon desensitization, unnecessary anxiety and confusion about meaning Risk to taking this patch: Low risk CSS removal and image deletion String or UUID changes made by this patch: None
Attachment #635867 -
Flags: approval-mozilla-aurora?
Updated•11 years ago
|
Attachment #635867 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Comment 6•11 years ago
|
||
Comment on attachment 635866 [details] [diff] [review] Remove Mixed State Icon - Beta *stamp*
Attachment #635866 -
Flags: approval-mozilla-beta? → approval-mozilla-beta+
Assignee | ||
Updated•11 years ago
|
Keywords: checkin-needed
Comment 7•11 years ago
|
||
Pushed to mozilla-inbound: https://hg.mozilla.org/integration/mozilla-inbound/rev/0880a8ce3407
status-firefox16:
--- → fixed
Flags: in-testsuite-
Keywords: checkin-needed
Target Milestone: Firefox 14 → Firefox 16
Version: Trunk → 14 Branch
Comment 8•11 years ago
|
||
Pushed to mozilla-beta: https://hg.mozilla.org/releases/mozilla-beta/rev/3b1c3202112f
Assignee: nobody → shorlander
Status: NEW → ASSIGNED
status-firefox14:
--- → fixed
status-firefox15:
--- → affected
Comment 9•11 years ago
|
||
Pushed to mozilla-aurora: https://hg.mozilla.org/releases/mozilla-aurora/rev/2107bc8dd678
Comment 10•11 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/0880a8ce3407
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Updated•11 years ago
|
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Target Milestone: Firefox 16 → ---
Version: 14 Branch → Trunk
Updated•11 years ago
|
Status: REOPENED → RESOLVED
Closed: 11 years ago → 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 16
You need to log in
before you can comment on or make changes to this bug.
Description
•