<div dir="ltr">Meta boxes are fine if they are specific to the theme, such as presentation-specific things that would only be used in the theme itself.</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Jun 3, 2014 at 1:05 PM, Imon Themes <span dir="ltr"><<a href="mailto:imon.theme@gmail.com" target="_blank">imon.theme@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi,<div>as new theme reviewer . i have question for  @<span style="font-family:arial,sans-serif;font-size:13px;font-weight:bold;white-space:nowrap">Emil Uzelac  @</span><span style="font-family:arial,sans-serif;font-size:13px;font-weight:bold;white-space:nowrap">Chip Bennett</span></div>

<div><span style="font-family:arial,sans-serif;font-size:13px;font-weight:bold;white-space:nowrap"><br></span></div><div><font face="arial, sans-serif"><span style="white-space:nowrap">i find issue in a theme . which is i currently reviewing . theme ticket is </span></font><a href="https://themes.trac.wordpress.org/ticket/18902" target="_blank">https://themes.trac.wordpress.org/ticket/18902</a><span style="white-space:nowrap;font-family:arial,sans-serif"> </span></div>

<div><br></div><div><font face="arial, sans-serif"><span style="white-space:nowrap">in this theme he use :</span></font></div><div><font face="arial, sans-serif"><span style="white-space:nowrap"><br></span></font></div><div>

<font face="arial, sans-serif"><span style="white-space:nowrap"><div>/*add meta_box*/</div><div> function cybergames_add_metabox_for_slider() {</div><div><span style="white-space:pre-wrap">    </span>add_meta_box( 'metabox_slider_id', 'Slider Post', 'cybergames_metabox_callback', 'post', 'side' );</div>

<div>}</div><div><br></div><div>/*callback function for meta box*/</div><div>function cybergames_metabox_callback() {</div><div><span style="white-space:pre-wrap"> </span>global $post; </div><div><span style="white-space:pre-wrap">  </span>$screen = get_current_screen(); ?></div>

<div><span style="white-space:pre-wrap">          </span><input type='checkbox' name='cbg_add_to_slider' id='cbg_add_to_slider' value='on' <?php if ( 'on' == get_post_meta( $post->ID, 'cbg_add_to_slider', true ) ) { ?> checked='checked' <?php } ?> /></div>

<div><span style="white-space:pre-wrap">          </span><label for='cbg_add_to_slider'><?php echo '&nbsp'. __( 'Display this', 'cybergames' ) .'&nbsp' . $screen->post_type . '&nbsp'. __( 'in slider', 'cybergames' ); ?></label></div>

<div><?php }</div><div><br></div><div><br></div></span></font></div><div><span style="font-family:arial,sans-serif;font-size:13px;font-weight:bold;white-space:nowrap">He use add meta box code . this code is acceptable or not ? </span></div>

<div><span style="font-family:arial,sans-serif;font-size:13px;font-weight:bold;white-space:nowrap"><br></span></div><div><span style="font-family:arial,sans-serif;font-size:13px;font-weight:bold;white-space:nowrap">thanks </span></div>

<table cellpadding="0" style="font-family:arial,sans-serif;font-size:13px"><tbody><tr><td style="width:587px"><br></td></tr></tbody></table></div><div class="gmail_extra"><div><div class="h5"><br><br><div class="gmail_quote">

On Sat, May 31, 2014 at 8:22 PM, Emil Uzelac <span dir="ltr"><<a href="mailto:emil@uzelac.me" target="_blank">emil@uzelac.me</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

great example :)<div><div><span></span><br><br>On Saturday, May 31, 2014, Chip Bennett <<a href="mailto:chip@chipbennett.net" target="_blank">chip@chipbennett.net</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">


<div dir="ltr">Right, here's the scenario that demonstrates the issue:<div><br></div><div>Developer A releases a free Theme that bundles the GPL3-licensed jQuery Isotope. Developer B takes Theme A, and uses it to create a derivative work, Theme B. Developer B then decides to sell Theme B.</div>



<div><br></div><div>According to the license terms of jQuery Isotope, Developer B must then purchase a commercial license (i.e. pay a fee) for an ostensibly GPL3-licensed library that came as part of Theme A.</div><div><br>



</div><div>That's not compatible with GPL.</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Sat, May 31, 2014 at 10:19 AM, Emil Uzelac <span dir="ltr"><<a>emil@uzelac.me</a>></span> wrote:<br>



<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">we discussed before terms are no good there, therefore not compatible<div><div><span></span><br>
<br>On Saturday, May 31, 2014, Justin Tadlock <<a>justin@justintadlock.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">


<div dir="ltr"><div>Isotope is released under two licenses, one is the GPL v3 in open source project. The other is a commercial license.  <a href="https://github.com/metafizzy/isotope" target="_blank">https://github.com/metafizzy/isotope</a></div>





<div><br></div><div><br></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Sat, May 31, 2014 at 8:44 AM, Emil Uzelac <span dir="ltr"><<a>emil@uzelac.me</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">thanks<div><div><span></span><br><br>On Saturday, May 31, 2014, Gary Barrett <<a>gary@headwaythemes.com</a>> wrote:<br>





<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div dir="ltr"><pre style="white-space:pre-wrap;color:rgb(0,0,0)">Hi everyone,

A theme called 'Bearded' is on the repo and is currently a featured theme. It contains jQuery Isotope, which
is released under a non-GPL compatible license.

The theme - <a href="https://themes.trac.wordpress.org/ticket/16610" target="_blank">http://wordpress.org/themes/bearded</a>

The script -
<a href="http://themes.svn.wordpress.org/sampression-lite/1.4.1.1/lib/js/jquery.isotope.min.js" target="_blank">http://themes.svn.wordpress.org/bearded/1.0.4/assets/js/libs/jquery.isotope.min.js</a>

Regards,
Gary</pre></div>
</blockquote>
</div></div><br>_______________________________________________<br>
theme-reviewers mailing list<br>
<a>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></blockquote></div><br></div>
</blockquote>
</div></div><br>_______________________________________________<br>
theme-reviewers mailing list<br>
<a>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></blockquote></div><br></div>
</blockquote>
</div></div><br>_______________________________________________<br>
theme-reviewers mailing list<br>
<a href="mailto:theme-reviewers@lists.wordpress.org" target="_blank">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></blockquote></div><br><br clear="all"><div><br></div></div></div><span class="HOEnZb"><font color="#888888">-- <br><div dir="ltr"><i>Imon Themes</i><div><br></div></div>
</font></span></div>
<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></blockquote></div><br></div>