Closed (fixed)
Project:
Drupal core
Version:
7.x-dev
Component:
overlay.module
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
26 Oct 2009 at 19:53 UTC
Updated:
31 Jan 2010 at 13:00 UTC
S6: at one point, we used to have a rounded close button without a drop shadow; we ended up with the image with drop shadow, although only compatible browsers will only display the drop shadow on the rest of the items, so the close button would stand out; we probably want to back out to the image and technique explained in http://drupal.org/node/517688#comment-2021274
Comments
Comment #1
seutje commentedI'm confused, he says to put the rounded corners in the image and then put rounded corners in the CSS as well? wouldn't that still be an issue for browsers that don't support any type of box-shadow or transparent PNGs or any type of border-radius like IE6
or would IE6 get a square box with no shadow? they're not really the 80% we're focussing on and it's a usable fallback
Comment #2
ksenzeeI think the idea is that older browsers won't recognize either the rounded corners or the drop shadow. They'll display the image as-is (rounded, with no shadow). Newer browsers will recognize the drop shadow, which will also show up as rounded, because they'll recognize the rounded corners as well. So in new browsers you'll get a rounded image with a rounded shadow and everyone is happy.
Comment #3
seutje commentedyea but the problem is that you can't make an image with rounded corners without using transparency, and since GIFs are ugly that leaves us with PNG, which isn't properly supported by some browsers so for them the area that's suppose to be transparent will be this ugly grey thing but I'll see what I can do
Comment #4
ksenzeeAh gotcha. I assume you're lumping 8-bit PNG (which is universally supported) in the same "ugly" category as GIFs, since the rounded edge would look a tad pixelated. Maybe an ugly 8-bit PNG where the rounded edge is slightly farther out than we want it? That way we could round it off in newer browsers and cover up the pixelation.
Comment #5
David_Rothstein commentedNo idea if this was fixed or not, so just moving to the correct queue for now.
Comment #6
aspilicious commentedThis is already fixed...
The images that you can find in the first link are the ones we are using now!