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)
Tracking
()
RESOLVED
FIXED
Firefox 2
People
(Reporter: lc__fc-xx001xx, Assigned: philor)
References
()
Details
(Keywords: fixed1.8.1)
Attachments
(3 files)
|
6.25 KB,
image/png
|
Details | |
|
955 bytes,
patch
|
mconnor
:
review+
mconnor
:
approval1.8.1+
|
Details | Diff | Splinter Review |
|
59.26 KB,
image/png
|
Details |
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
| Assignee | ||
Updated•19 years ago
|
Status: UNCONFIRMED → NEW
Ever confirmed: true
Flags: blocking-firefox2?
Target Milestone: --- → Firefox 2
Version: unspecified → 2.0 Branch
| Assignee | ||
Comment 2•19 years ago
|
||
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?
| Assignee | ||
Comment 3•19 years ago
|
||
Obligatory screenshot.
Comment 4•19 years ago
|
||
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+
Comment 5•19 years ago
|
||
1.8
mozilla/browser/themes/pinstripe/browser/safebrowsing/browser-protection.css 1.1.2.8
Updated•19 years ago
|
Flags: blocking-firefox2? → blocking-firefox2+
You need to log in
before you can comment on or make changes to this bug.
Description
•