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

[michael at mfields.org]

One thing that I noticed that is totally unrelated is the use of:

@package WordPress

In the file's docblock. The Inline Documentation codex page states the
"Plugin authors must not use the @package WordPress in their plugins.".
Seems like a rule that should apply to anything outside of core. Just
wondering what everyone on the list thought.

http://codex.wordpress.org/Inline_Documentation#Plugin_.40Package_Tag

> 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
>