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)

x86
Linux
defect

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
Attached image screenshot
Assignee: nobody → mark.finkle
Flags: blocking-fennec1.0+
Priority: -- → P2
Target Milestone: --- → Fennec A1
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)
Attachment #340069 - Flags: review?(gavin.sharp) → review+
Attachment #340069 - Flags: ui-review? → ui-review?(madhava)
Attachment #340069 - Flags: ui-review?(madhava)
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+
Keywords: uiwanted
Madhava gave his blessing, so I marking this fixed.
Status: NEW → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Verified the new styled Larry with a 10/2 build.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: