[theme-reviewers] Base64 Used to Encode Transparent Images?!?

[Curtis Work]

I'm not sire what the official position on browser support is but if IE7 is in there then they should also be including those images in an IE7 stylesheet since it doesn't support data uri. 

Curtis McHale
Http://www.curtismchale.com
604.751.3482

On 2010-11-16, at 8:34, Chip Bennett <chip at chipbennett.net> wrote:

> Okay, then. Good enough for me!
> 
> I'll approve the ticket. But, I'd like someone with more PHP/security expertise than I have to take a look at what this Theme is doing in admin/ with overwriting files and changing permissions and whatnot, just to make sure there aren't any issues. I *think* everything is okay, but following all of that is still a bit above my pay grade.
> 
> Chip
> 
> On Tue, Nov 16, 2010 at 10:28 AM, Otto <otto at ottodestruct.com> wrote:
> On Tue, Nov 16, 2010 at 10:24 AM, Chip Bennett <chip at chipbennett.net> wrote:
> > So, essentially, the image itself is embedded in the CSS file?
> > I'm still uneasy about it, but that could be due to lack of understanding.
> > And I still question whether it's actually necessary.
> 
> Could he put the image file in a separate file and then make an http
> link to it there instead? Sure.
> Is including the data inline like this faster? Yes, it's much, much faster.
> 
> For small images (these are 1x1 pixel PNG images), this is WAY faster
> than including the image separately URL and incurring the overhead of
> an extra HTTP request.
> 
> Necessary? No.
> Understandable and faster and better and safe and actually the correct
> way to do it? Yes.
> 
> I give this a total thumbs up and an "okay". There's nothing wrong
> here of any sort.
> 
> -Otto
> _______________________________________________
> theme-reviewers mailing list
> theme-reviewers at lists.wordpress.org
> http://lists.wordpress.org/mailman/listinfo/theme-reviewers
> 
> _______________________________________________
> theme-reviewers mailing list
> theme-reviewers at lists.wordpress.org
> http://lists.wordpress.org/mailman/listinfo/theme-reviewers
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.wordpress.org/pipermail/theme-reviewers/attachments/20101116/01a3d064/attachment.htm>