Closed
Bug 456149
Opened 16 years ago
Closed 16 years ago
Larry is naked when SSL information is displayed
Categories
(Firefox for Android Graveyard :: General, defect, P2)
Tracking
(Not tracked)
VERIFIED
FIXED
fennec1.0a1
People
(Reporter: abillings, Assigned: mfinkle)
Details
(Keywords: uiwanted)
Attachments
(3 files, 1 obsolete file)
If a user goes to a site that is encrypted and selects the favicon to show Larry and site information, the text is drawn without a background, making it impossible to read in most instances. This was seen in Stuart's build 20080918185520. Steps to Reproduce 1. Go to http://www.verisign.com 2. Select the favicon Result: Poor Naked Larry
Reporter | ||
Comment 1•16 years ago
|
||
Updated•16 years ago
|
Assignee: nobody → mark.finkle
Flags: blocking-fennec1.0+
Priority: -- → P2
Target Milestone: --- → Fennec A1
Assignee | ||
Comment 2•16 years ago
|
||
This patch uses the OS X style theming for the identity panel instead of letting the OS theme the panel. We could pick a different color and transparency, but this is a start
Attachment #340069 -
Flags: ui-review?
Attachment #340069 -
Flags: review?(gavin.sharp)
Updated•16 years ago
|
Attachment #340069 -
Flags: review?(gavin.sharp) → review+
Assignee | ||
Updated•16 years ago
|
Attachment #340069 -
Flags: ui-review? → ui-review?(madhava)
Assignee | ||
Updated•16 years ago
|
Attachment #340069 -
Flags: ui-review?(madhava)
Assignee | ||
Comment 3•16 years ago
|
||
Same as last patch, but the colors match to current theme and I removed unused CSS. Carrying gavin's r+ I'm landing this patch, but not resolving the bug so Madhava will be able see the theme and make suggestions. This is certainly better than a naked larry
Attachment #340069 -
Attachment is obsolete: true
Attachment #341063 -
Flags: review+
Assignee | ||
Comment 4•16 years ago
|
||
http://hg.mozilla.org/mobile-browser/rev/f4ee6a1d188c
Assignee | ||
Comment 5•16 years ago
|
||
Assignee | ||
Comment 6•16 years ago
|
||
Madhava gave his blessing, so I marking this fixed.
Status: NEW → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Reporter | ||
Comment 7•16 years ago
|
||
Verified the new styled Larry with a 10/2 build.
Reporter | ||
Updated•16 years ago
|
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•