When text-zooming the page or if a theme defines a bigger base font, the toolbar buttons look ugly because the image sprite used does not define a big enough area for their backgrounds.

Since there are already 2 issues around the toolbar image, #535066: Use CSS3 / IE filter to render toolbar shadow and #703490: Use PNG24 images that degrade to PNG8 on IE6, this is not worth committing now, but I created the issue anyway so we can come back to it later. Thought it was better to keep this issue distinct. This patch removes the shadow in the meanwhile.

Comments

jide’s picture

StatusFileSize
new591 bytes
new2.09 KB
new8.7 KB
new8.32 KB

Files got lost.

jbrown’s picture

Status: Postponed » Closed (duplicate)

This is a duplicate: #667918: Sprite height too short causing active state graphical bug in toolbar

Anyway, this is not the way to fix it. There is no point in just making more space in the image, as it will always be possible to zoom in more. Also, it looks weird with the gradient just ending before the bottom.

jide’s picture

Thanks for the duplicate, did not catch this one.
I think text zooming is considered accessibility friendly when user can zoom up to 200%, so I think a bigger background covers enough zoom levels to be worth it.
The gradient is exactly the same, i just added more plain gray at the bottom of it.

Let's continue this on the other thread anyway.