Minor improvements to the disabled Forward and Stop buttons in Classic. The bottom row of the disabled Forward button isn't hatched, and the hatching in the disabled Stop button breaks pattern, causing banding. The replacement images were made from the originals so they are the same size and color depth, have the same palette, and the icons are positioned correctly within them. Only the absolute minimum required was changed. 2002-02-04-06 Win98 (0.9.8 milestone)

confirming for consideration

Comment on attachment 67997 [details] Fixed chrome://navigator/skin/icons/btn1.gif r=andreww just make sure that your changes dont change the pixel offsets of the images so that the -moz-image-regions dont get off track

Comment on attachment 67998 [details] Fixed chrome://communicator/skin/icons/btn1.gif r=andreww

I worked from the originals and was careful about not changing anything else. They're okay in that respect.

andrew, can you take this?

ok - though this may need a new reviewer now :)

Your review should still be valid considering you didn't create the replacement images.

hewitt - can you sr?

sr=hewitt

ok greg are the gif's you posted here the ones you want me to check in, or do you want to attach a zip file with the gifs together?

The two gifs attached can go in as is.

Just curious, but why Mozilla 1.0? Why can't this go in now?

FYI: see also bug 125526 which wants to fix the weird gray stuff in the navigation toolbar icons.

Ill try to check this in today.

Im guessing 125526 superceeds this one since it incorporates this image's changes. Will get that one checked in.

andreww, please do check in the second gif from this bug (chrome://communicator/skin/icons/btn1.gif). It is not covered in bug 125526, I only stole (and tried to improve) the first gif from this bug.

yep - I understand. I'll check in the nav one from the other bug and the communicator one from this bug.

fix checked in.

Verified (2002-02-21-08-trunk)