480597 - Recut CTA graphics, make Learn More graphic work in IE 6

Status: VERIFIED FIXED

(Websites Graveyard :: spreadfirefox.com, defect)

Description

[Neil Lee [:neilio]]

New bug to track multiple fixes to the CTA area.

Comment 1

[Neil Lee [:neilio]]

Attachment: Improved CTA graphics, redid the learn more graphic so it renders correctly in IE 6 (old one was PNG24), and some alignment fixes

Comment 2

[Neil Lee [:neilio]]

Attachment: New images referenced by this patch

Comment 3

[Alex Buchanan [:abuchanan]]

Comment on attachment 364561 [details] [diff] [review]
Improved CTA graphics, redid the learn more graphic so it renders correctly in IE 6 (old one was PNG24), and some alignment fixes

everything looks OK to me

Comment 4

[Alex Buchanan [:abuchanan]]

Comment on attachment 364561 [details] [diff] [review]
Improved CTA graphics, redid the learn more graphic so it renders correctly in IE 6 (old one was PNG24), and some alignment fixes

actually, sorry I confused myself.  The Learn more link looks screwy on FF3 for me.  screenshot to follow

Comment 5

[Alex Buchanan [:abuchanan]]

Attachment: new image funk

this shows my dev homepage with patches from attachments 364561 and 364563 applied.  The Learn More images are funky and if you look closely, there is a small, vertical, white line to the left of the Learn More in the Spread block

Comment 6

[Neil Lee [:neilio]]

Attachment: Fixed image

Ah, good eye! This image fixes the issue.

Comment 7

[Neil Lee [:neilio]]

Note that the Learn More graphic isn't going to be the clearest because it needs to work in IE, so I couldn't use a PNG24. The only other way to fix this is to have separate Learn More images for each box with that box's background colour, which is kind of overkill imo.

Comment 8

[Alex Buchanan [:abuchanan]]

(In reply to comment #7)
> Note that the Learn More graphic isn't going to be the clearest because it
> needs to work in IE, so I couldn't use a PNG24. The only other way to fix this
> is to have separate Learn More images for each box with that box's background
> colour, which is kind of overkill imo.

The new learn more images are really shotty though.  Not sure what the best solution is, but the new images won't do.  

Can we get better quality without using PNG24?  I assume the underlying problem is that IE 6 doesn't support PNG transparency?

Can we do this with text only?  Drop the learn more images altogether?

I agree, having different images per background color is lame, but if it get's everyone the best quality for now (on our short schedule) until we figure out how to do it better (or drop IE6), and we only have to add 4 images, that sounds like the best approach to me.

There is a JS hack for IE/PNG transparency, although that sounds time-costly.

Comment 9

[Mary]

Pardon my ignorance here, but is this anything that TRO can help with?  If so, let me know and we can get something turned around asap!

Comment 10

[Alex Buchanan [:abuchanan]]

Neil, if you recut the learn more images, I'll commit the patch and new images in one go.

Thanks!

Comment 11

[Neil Lee [:neilio]]

Attachment: New Learn More images and patch

Okay, here's the new images and patch to make the Learn More text less craptacular-looking.

Comment 12

[Neil Lee [:neilio]]

Attachment: Another attempt at the learn more images

Just the images this time (the last patch is still relevant). Any better?

Comment 13

[Alex Buchanan [:abuchanan]]

Comment on attachment 368438 [details]
Another attempt at the learn more images

i think i committed these in r23656.  is this resolved fixed now?

Comment 14

[Stephen Donner [:stephend] Not actively reading bugmail]

(In reply to comment #13)
> (From update of attachment 368438 [details])
> i think i committed these in r23656.  is this resolved fixed now?

It's looking a lot better, but we're still clipping the bottom of p, q, j, g letters in IE 6, but that can be spun off.

Comment 15

[Neil Lee [:neilio]]

This is mostly fixed - the clipping in IE is a side effect of how these images need to be cut to work cross-browser, so it's ultimately a minor issue I don't think needs revisiting.

Comment 16

[Stephen Donner [:stephend] Not actively reading bugmail]

Fine by me -- at this point, it's something I'm totally happy to live with, too.

Verified FIXED.