Toolbar.png needs to be cropped

VERIFIED FIXED in Firefox 2 alpha1

Status

()

--
trivial
VERIFIED FIXED
13 years ago
12 years ago

People

(Reporter: Gavin, Assigned: philip.puryear)

Tracking

({verified1.8.1})

Trunk
Firefox 2 alpha1
verified1.8.1
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(4 attachments, 2 obsolete attachments)

In bug 306453, the toolbar item for "Read Mail" was removed, so now Toolbar.png in both winstripe and pinstripe contain an unused icon. It should just be cropped, removing the mail icon. No CSS changes are needed since the icon is the left-most icon in the set.
(Assignee)

Comment 1

13 years ago
Created attachment 208700 [details]
Toolbar.png (pinstripe)

Something like this?

(I can't assign bugs to myself, otherwise I'd do so.)
Attachment #208700 - Flags: review?(gavin.sharp)
(Assignee)

Comment 2

13 years ago
Created attachment 208701 [details]
Toolbar.png (winstripe)
Attachment #208701 - Flags: review?(gavin.sharp)
Comment on attachment 208700 [details]
Toolbar.png (pinstripe)

Yes, that's the idea, except that the pinstripe image should be 448 pixels wide (32px per "button"), and the winstripe one should be 336 pixels wide (24 px per "button"). I also noticed that the Toolbar-small.png images need to be changed as well. I also wonder if there are any special considerations to be made to ensure no loss of quality/transparency/etc when modifying the images (as noted, the editor I had used messed the icons up considerably).
Attachment #208700 - Flags: review?(gavin.sharp) → review-
Attachment #208701 - Flags: review?(gavin.sharp) → review-
(Assignee)

Comment 4

13 years ago
When done in Photoshop, transparency shouldn't be an issue, but bicubic resizing of an image will result in a little loss of quality. If that's not an issue, I could just resize the entire image, but if it is I suppose it could be re-pieced together from scratch. Are all those images available somewhere in a higher resolution?
Why do you need to resize the image? If just cropping it at those dimensions cuts off one of the images then something funky is going on :). I'm assuming all the images are of a fixed width... if they're not, this bug is a little harder to fix, since you'd need to check all the CSS-specified coordinates and see what area each button is using, but I doubt that is the case.

And thanks for taking a look at this!
(Assignee)

Comment 6

13 years ago
Oh! At first I thought you meant that I needed to somehow reconstruct the entire image from scratch, rather than that mine were just a few pixels off. I'll attach correct ones shortly.
(Assignee)

Updated

13 years ago
Attachment #208700 - Attachment is obsolete: true
(Assignee)

Updated

13 years ago
Attachment #208701 - Attachment is obsolete: true
(Assignee)

Comment 7

13 years ago
Created attachment 208705 [details]
Toolbar.png (pinstripe) v2
Attachment #208705 - Flags: review?(gavin.sharp)
(Assignee)

Comment 8

13 years ago
Created attachment 208706 [details]
Toolbar.png (winstripe) v2
Attachment #208706 - Flags: review?(gavin.sharp)
(Assignee)

Comment 9

13 years ago
What are the required dimensions for Toolbar-small.png?
I just took the current image widths and divided by the current number of icons: so 240/15=16 and 360/15=24 for Winstripe and Pinstripe, respectively.
(Assignee)

Comment 11

13 years ago
Created attachment 208707 [details]
Toolbar-small.png (pinstripe)
Attachment #208707 - Flags: review?(gavin.sharp)
(Assignee)

Comment 12

13 years ago
Created attachment 208708 [details]
Toolbar-small.png (winstripe)
Attachment #208708 - Flags: review?(gavin.sharp)
Thanks Philip, I will take a look at these and test tomorrow.
Attachment #208708 - Flags: review?(gavin.sharp) → review+
Attachment #208706 - Flags: review?(gavin.sharp) → review+
Attachment #208707 - Flags: review?(gavin.sharp) → review+
Attachment #208705 - Flags: review?(gavin.sharp) → review+
I tested these on Windows, Jesse tested on Mac, eveything looks good. I will mail the theme author to double-check that this is OK before landing this. Thanks again Philip!

Comment 15

13 years ago
Looks good. Thanks for letting me know Gavin!
Assignee: nobody → philippuryear
(Assignee)

Comment 16

13 years ago
Now that the patch for bug 306453 has been approved for 1.8, should I seek similar approval for these images so they can land on the branch along with that?

Also, I noticed that this was checked in yesterday. Can I mark this FIXED yet?
Oops, sorry, I forgot to mark the bug fixed when I checked it in. I'll land this on the branch when I land the patch in bug 306453.
Status: NEW → RESOLVED
Last Resolved: 13 years ago
Resolution: --- → FIXED
Landed this and bug 306453 on the 1.8 branch for Firefox 2.
Keywords: fixed1.8.1
Target Milestone: --- → Firefox 2
Target Milestone: Firefox 2 → Firefox 2 alpha1
Status: RESOLVED → VERIFIED
Keywords: fixed1.8.1 → verified1.8.1
You need to log in before you can comment on or make changes to this bug.