Closed Bug 175515 Opened 22 years ago Closed 22 years ago

Bookmarks Toolbar Icons are corrupt

Categories

(Firefox :: Bookmarks & History, defect)

x86
Windows XP
defect
Not set
trivial

Tracking

()

VERIFIED FIXED
Phoenix0.4

People

(Reporter: tobytinkerman, Assigned: p_ch)

Details

Attachments

(1 file)

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.2b) Gecko/20021019 Phoenix/0.3
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.2b) Gecko/20021019 Phoenix/0.3

Since the 10-18 build, icons in the Bookmarks Toolbar seem shrunk/corrupt -
especially the default bookmark icon.

Reproducible: Always

Steps to Reproduce:
confired. I see this on linux. It looks like they're being scaled down a pixel
or two. 
Assignee: hyatt → chanial
Status: UNCONFIRMED → NEW
Component: Toolbars → Bookmarks
Ever confirmed: true
Attached patch PatchSplinter Review
Here's the patch to fix this. The bug was caused by this commit:
http://bonsai.mozilla.org/cvsview2.cgi?diff_mode=context&whitespace_mode=show&subdir=mozilla/browser/base/skin&command=DIFF_FRAMESET&file=browser.css&rev1=1.64&rev2=1.65&root=/cvsroot


As a temporary workaround, you can add the following css to your
userChrome.css:

toolbar[mode="icons"] .toolbarbutton-icon {
  padding: 0px 0px 0px 0px !important;
}
Fixed.
Status: NEW → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
Many thanks; as far as I can tell, the preceding workaround performs
wonderfully, at least for all the sites I bookmark, save one: www.kuro5hin.org.
 It appears that most sites use a 16x16 favicon; kuro5hin uses a 32x32 icon. 
The icon renders perfectly in a Tab; when it is scaled for the Toolbar, and for
the 'Bookmarks' menu item, it retains the '1 or 2 pixel' scrunchy problem; and
in the 'Bookmarks' sidebar, it is cropped to the upper-left quadrant, giving a
'quarter-round' image.
Target Milestone: --- → Phoenix0.4
VERIFIED Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.5a) Gecko/20030714
Mozilla Firebird/0.6
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: