<div dir="ltr"><div>Emil,</div><div><br></div><div>I thought it'd be better to discuss an issue with other reviewers before pointing it out to the theme author to prevent confusion. It appeared directly discussing it on the ticket would be better. Noted for future reference.</div><div><br></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Sep 10, 2014 at 7:19 AM, 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"><div dir="ltr"><div class="gmail_default" style="font-family:arial,helvetica,sans-serif">Made notes in the ticket directly!</div></div><div><div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Sep 9, 2014 at 7:13 PM, Fikri Rasyid <span dir="ltr"><<a href="mailto:fikrirasyid@gmail.com" target="_blank">fikrirasyid@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">Edward, <div><br></div><div>I agree that it would be perfect if two authors work together on the original theme to make it compatible. I haven't suggested it tho, should I suggest it and pending the reviewing process of this theme?</div><div><br></div><div>Regarding the forking issue above, there are more things i'd love to clarify regarding forking a theme.</div><div><br></div><div>Chip stated on this guideline (a bit old, tho) <a href="http://www.chipbennett.net/2011/04/20/a-guide-to-reviewing-themes-for-the-wordpress-theme-repository/" target="_blank">http://www.chipbennett.net/2011/04/20/a-guide-to-reviewing-themes-for-the-wordpress-theme-repository/</a> that:</div><div><br></div><div><b><i>All custom Theme functions, constants, classes, global variables, and options are prefixed with theme-slug (or an appropriate variant).</i></b><br></div><div><br></div><div>However, due to the nature of forking the theme, there are lots of function prefixes and text domains that are still refer to the original theme. The issue would be:</div><div><br></div><div>Is it okay to leave function prefix it as is or should one change the function prefix to its unique theme slug when forking a theme (and submitting it to the repository)? I'm about to ask the same question about the text domain but it is super obvious that text domain should be unique to the theme slug</div></div><div><div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Sep 10, 2014 at 6:58 AM, Edward Caissie <span dir="ltr"><<a href="mailto:edward.caissie@gmail.com" target="_blank">edward.caissie@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">The argument of not being able to create a Grandchild-Theme is very strong so from that aspect providing proper attribution to the original theme is given the idea of forking a theme is fine.<div><br></div><div>As to this specific theme and forking it to be specifically compatible with a plugin, I would expect there to be much more to the "new" theme than just some plugin compatibility code ... has it been suggested the two authors work together on the original theme to make it compatible with the plugin then building Child-Themes as the forking author intends would be more easily accomplished.</div></div><div class="gmail_extra"><br clear="all"><div>Edward Caissie<br>aka Cais.</div>
<br><div class="gmail_quote"><div><div>On Tue, Sep 9, 2014 at 7:33 PM, Fikri Rasyid <span dir="ltr"><<a href="mailto:fikrirasyid@gmail.com" target="_blank">fikrirasyid@gmail.com</a>></span> wrote:<br></div></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div><div dir="ltr">Hi all,<div><br></div><div>I'm currently reviewing this theme <a href="https://themes.trac.wordpress.org/ticket/20060" target="_blank">https://themes.trac.wordpress.org/ticket/20060</a> and a theme @pseudoxiah made me aware that that theme is 'a fork' of another theme. The diff can be seen here:<a href="https://themes.trac.wordpress.org/changeset?old_path=/pinboard/1.1.6&new_path=/clickhome-myhome/1.1" target="_blank">https://themes.trac.wordpress.org/changeset?old_path=/pinboard/1.1.6&new_path=/clickhome-myhome/1.1</a></div><div><br></div><div>The theme author argues that the reason for him to fork this instead of creating a child theme is that he wants user to further create a child theme based on this theme and this theme is made to be compatible and used with specific plugin (<a href="https://wordpress.org/plugins/clickhome-myhome/" target="_blank">https://wordpress.org/plugins/clickhome-myhome/</a>)</div><div><br></div><div>I honestly think that it is okay to fork and submit a fork as long as the theme gives appropriate credit to the original theme. However, I'm still new at this and wonder is there any specific guideline about forking a theme and submit it to the repo?</div><div><br></div><div>Thank you.</div><span><font color="#888888"><div><br clear="all"><div><br></div>-- <br>Best Regards,<br><br><br>Fikri Rasyid<br><a href="http://fikrirasyid.com" target="_blank">http://fikrirasyid.com</a>
</div></font></span></div>
<br></div></div>_______________________________________________<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></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>-- <br>Best Regards,<br><br><br>Fikri Rasyid<br><a href="http://fikrirasyid.com" target="_blank">http://fikrirasyid.com</a>
</div>
</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></div>
</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>-- <br>Best Regards,<br><br><br>Fikri Rasyid<br><a href="http://fikrirasyid.com" target="_blank">http://fikrirasyid.com</a>
</div></div>