235190 - Cookie privacy icon changes height of status bar (classic theme)

Status: RESOLVED FIXED

(Core :: Networking: Cookies, defect)

Description

[HJ]

The height of the status changes as soon as the privacy icon is displayed.

Steps to reproduce:
1) accepts cookies based on privacy settings
2) visit cnn.com

Current result: see attached screenshot

Comment 1

[HJ]

Attachment: Enlarged screenshot of statusbar (250%)

Comment 2

[HJ]

Proposed changes:

http://lxr.mozilla.org/seamonkey/source/themes/classic/navigator/navigator.css#344
http://lxr.mozilla.org/seamonkey/source/themes/modern/navigator/navigator.css

+#privacy-button {
+  list-style-image: url("chrome://communicator/skin/cookie/taskbar-cookie.gif");
+  padding-right: 2px;
+}

http://lxr.mozilla.org/seamonkey/source/extensions/cookie/resources/content/cookieTasksOverlay.xul#117

-                       oncommand="viewP3PDialog()"
tooltiptext="&cookieIcon.label;">
+                       oncommand="viewP3PDialog()"
tooltiptext="&cookieIcon.label;"/>
-         <toolbarbutton class="taskbutton" id="mini-cookie"
+                       
image="chrome://communicator/skin/cookie/taskbar-cookie.gif"/>

Comment 3

[HJ]

Oops, replace the last + with -

Comment 4

[[e-mail]]

Attachment: Comment 2 as a patch

Comment 5

[[e-mail]]

Comment on attachment 142080 [details] [diff] [review]
Comment 2 as a patch

p=hj, r=me, sr=darin?

Comment 6

[HJ]

Thanks Neil. I am still far away from home and we're moving another 3500 miles
next Friday, so still no CVS access for another X weeks.

Comment 7

[[e-mail]]

Fix checked in.

Comment 8

[Alfred Kayser]

*** Bug 228566 has been marked as a duplicate of this bug. ***