Closed Bug 352945 Opened 19 years ago Closed 19 years ago

[mac] with phishing protection warning icon the location bar gets 2px higher and a part of the contour vanishes

Categories

(Firefox :: Address Bar, defect)

2.0 Branch
PowerPC
macOS
defect
Not set
normal

Tracking

()

RESOLVED FIXED
Firefox 2

People

(Reporter: lc__fc-xx001xx, Assigned: philor)

References

()

Details

(Keywords: fixed1.8.1)

Attachments

(3 files)

User-Agent: Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:1.8.1) Gecko/20060915 BonEcho/2.0 Build Identifier: Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:1.8.1) Gecko/20060915 BonEcho/2.0 When the phishing protection warning icon appears in the location bar it gets 2 px higher and apparently because of that a part of its contour vanishes, which seems to be covered by the safe browsing popup tail now. Reproducible: Always Steps to Reproduce: 1. go to http://www.google.com/tools/firefox/safebrowsing/phish-o-rama.html 2. look at location bar Actual Results: The location bar gets 2 px higher and a part of its contour vanishes, apparently covered by the safe browsing popup tail. Expected Results: The location bar should stay as high as always and its contour should by displayed correctly.
Attachment #238768 - Attachment description: screenshot fo the problem → screenshot of the problem
Status: UNCONFIRMED → NEW
Ever confirmed: true
Flags: blocking-firefox2?
Target Milestone: --- → Firefox 2
Version: unspecified → 2.0 Branch
Attached patch BandaidSplinter Review
This isn't perfect, but at least it's extremely minimal, and unbreaks things. The tail comes out of the right side of the icon, instead of the middle, but that still strikes me as better than changing the size of the whole addressbar.
Assignee: nobody → philringnalda
Status: NEW → ASSIGNED
Attachment #238936 - Flags: review?(mconnor)
Attachment #238936 - Flags: approval1.8.1?
Attached image "After" screenshot
Obligatory screenshot.
Comment on attachment 238936 [details] [diff] [review] Bandaid I, uh, feel dumb. I fixed this for the security icon last week, and forgot to check this.
Attachment #238936 - Flags: review?(mconnor)
Attachment #238936 - Flags: review+
Attachment #238936 - Flags: approval1.8.1?
Attachment #238936 - Flags: approval1.8.1+
1.8 mozilla/browser/themes/pinstripe/browser/safebrowsing/browser-protection.css 1.1.2.8
Status: ASSIGNED → RESOLVED
Closed: 19 years ago
Keywords: fixed1.8.1
Resolution: --- → FIXED
Flags: blocking-firefox2? → blocking-firefox2+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: