<div dir="ltr">Email me off-list. We (admins) will facilitate the contact.</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Mar 26, 2014 at 9:22 AM, Konstantin Kovshenin <span dir="ltr"><<a href="mailto:kovshenin@gmail.com" target="_blank">kovshenin@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Masonry v2 is a jQuery plugin, so it's jquery-masonry which has been<br>
in core for over a year. Not sure why themes still bundle their own<br>
script, maybe they're super old, haven't checked. Masonry v3's handle<br>
is just masonry because it doesn't use jQuery anymore. jquery-masonry<br>
became the shim that provides limited backwards compatibility with v2<br>
and has a jQuery dependency.<br>
<br>
So if themes used core's jquery-masonry before 3.9 they're probably<br>
going to be fine. If they bundled their own jquery masonry and called<br>
it masonry, there's a good chance they will break :)<br>
<br>
What is the best way to reach out to the authors of the following<br>
themes (rather than scraping their e-mail addresses and contact forms<br>
by hand)?<br>
<br>
<a href="http://pastebin.com/jaaRqjiS" target="_blank">http://pastebin.com/jaaRqjiS</a><br>
<br>
Thanks!<br>
<div class="HOEnZb"><div class="h5"><br>
On Wed, Mar 26, 2014 at 5:10 PM, Chip Bennett <<a href="mailto:chip@chipbennett.net">chip@chipbennett.net</a>> wrote:<br>
> I would recommend working directly with the developers of the 20 Themes.<br>
><br>
> This situation does help clarify what our guidelines should be regarding<br>
> Theme-bundled, third-party scripts. I think the Guidelines supported this<br>
> already, but: only *custom* scripts bundled with the Theme should use a<br>
> theme-prefixed handle. A third-party script should use a non-prefixed<br>
> handle.<br>
><br>
> Is there a nomenclature standard for such scripts? e.g. Would Masonry's<br>
> handle be 'masonry' or 'jquery-masonry', etc.?<br>
><br>
><br>
> On Wed, Mar 26, 2014 at 8:50 AM, Konstantin Kovshenin <<a href="mailto:kovshenin@gmail.com">kovshenin@gmail.com</a>><br>
> wrote:<br>
>><br>
>> That's if both pieces of code try to load the script, yes, but core's<br>
>> situation is a bit different.<br>
>><br>
>> We'll end up loading (plain, non-jQuery) Masonry v3 when a theme used<br>
>> that handle to load its own jQuery Masonry v2. If the theme uses a<br>
>> prefixed themename-masonry handle it will keep working.<br>
>><br>
>> Basically what I'm asking is for Otto to weigh in on<br>
>> <a href="https://core.trac.wordpress.org/ticket/27510" target="_blank">https://core.trac.wordpress.org/ticket/27510</a> :)<br>
>><br>
>> Maybe we can change core's handle for the sake of the 20 or so<br>
>> affected themes, maybe we can notify the authors, or maybe we can just<br>
>> fix it and let reviewers know going forward.<br>
>><br>
>> On Wed, Mar 26, 2014 at 4:34 PM, Otto <<a href="mailto:otto@ottodestruct.com">otto@ottodestruct.com</a>> wrote:<br>
>> > On Wed, Mar 26, 2014 at 1:26 AM, Konstantin Kovshenin<br>
>> > <<a href="mailto:kovshenin@gmail.com">kovshenin@gmail.com</a>><br>
>> > wrote:<br>
>> >><br>
>> >> Here's a follow-up question.<br>
>> >><br>
>> >> Given that a theme has used a non-prefixed generic name for a bundled<br>
>> >> script, how could we ensure it doesn't break when core introduces the<br>
>> >> same handle but with a different version? Masonry v3 in 3.9 for<br>
>> >> example :)<br>
>> ><br>
>> ><br>
>> ><br>
>> > If two bits of code loaded the same script with two different names,<br>
>> > then<br>
>> > they'd both load, and that would most likely cause breakage too. So you<br>
>> > can't win here.<br>
>> ><br>
>> > Best to choose the most obvious name based on the source of the script,<br>
>> > so<br>
>> > that hopefully both of them don't load and only one of them does. This<br>
>> > may<br>
>> > break, but it's less likely to break than loading two different versions<br>
>> > of<br>
>> > the same script is.<br>
>> ><br>
>> > -Otto<br>
>> ><br>
>> ><br>
>> > _______________________________________________<br>
>> > theme-reviewers mailing list<br>
>> > <a href="mailto:theme-reviewers@lists.wordpress.org">theme-reviewers@lists.wordpress.org</a><br>
>> > <a href="http://lists.wordpress.org/mailman/listinfo/theme-reviewers" target="_blank">http://lists.wordpress.org/mailman/listinfo/theme-reviewers</a><br>
>> ><br>
>><br>
>><br>
>><br>
>> --<br>
>> Konstantin<br>
>> _______________________________________________<br>
>> theme-reviewers mailing list<br>
>> <a href="mailto:theme-reviewers@lists.wordpress.org">theme-reviewers@lists.wordpress.org</a><br>
>> <a href="http://lists.wordpress.org/mailman/listinfo/theme-reviewers" target="_blank">http://lists.wordpress.org/mailman/listinfo/theme-reviewers</a><br>
><br>
><br>
><br>
> _______________________________________________<br>
> theme-reviewers mailing list<br>
> <a href="mailto:theme-reviewers@lists.wordpress.org">theme-reviewers@lists.wordpress.org</a><br>
> <a href="http://lists.wordpress.org/mailman/listinfo/theme-reviewers" target="_blank">http://lists.wordpress.org/mailman/listinfo/theme-reviewers</a><br>
><br>
<br>
<br>
<br>
--<br>
Konstantin<br>
_______________________________________________<br>
theme-reviewers mailing list<br>
<a href="mailto:theme-reviewers@lists.wordpress.org">theme-reviewers@lists.wordpress.org</a><br>
<a href="http://lists.wordpress.org/mailman/listinfo/theme-reviewers" target="_blank">http://lists.wordpress.org/mailman/listinfo/theme-reviewers</a><br>
</div></div></blockquote></div><br></div>