<div dir="ltr">Definitely agree with that one.<br></div><div class="gmail_extra"><br><br><div class="gmail_quote">2013/11/22 Chip Bennett <span dir="ltr"><<a href="mailto:chip@chipbennett.net" target="_blank">chip@chipbennett.net</a>></span><br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">That could certainly be added for register_post_type() and register_taxonomy(), but add_meta_box() isn't inherently improper; it all depends on its use.</div>
<div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><br><div class="gmail_quote">
On Fri, Nov 22, 2013 at 8:08 AM, Towfiq I. <span dir="ltr"><<a href="mailto:tislam100@gmail.com" target="_blank">tislam100@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">when we submit a theme through the uploader. The uploader detects and returns an error when there is a important tag missing. Isn't it possible to make the uploader extend its functionality and check whether the uploaded theme has custom post type registered with register_post_type and if it does, the uploader shows an error and does not continue with the submission process.<div>
<br></div><div>just my 2 bits. </div><div><br></div><div><br></div></div><div class="gmail_extra"><div><div><br><br><div class="gmail_quote">On Fri, Nov 22, 2013 at 6:34 PM, Chip Bennett <span dir="ltr"><<a href="mailto:chip@chipbennett.net" target="_blank">chip@chipbennett.net</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">In this specific instance, I wonder if this isn't a legacy of the in-core-then-removed-in-beta Post Formats UI post-meta handling of post format-specific data? If that's the case, I have less of a concern with the use of custom post meta here - but I'd like to get some clarification from the core devs to know, when the feature is addressed again in the future, if the same post-meta implementation will be used.</div>
<div><div>
<div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Nov 22, 2013 at 4:38 AM, Rohit Tripathi <span dir="ltr"><<a href="mailto:rohitink@live.com" target="_blank">rohitink@live.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div><div dir="ltr">Theme specific design is acceptable. But Enterting MP3 Urls for audio format, and Video Embed Code for Video Formats is not acceptable. This is what is done in the themes i mentioned.<br><br><div><hr>
Date: Fri, 22 Nov 2013 10:37:19 +0100<br>From: <a href="mailto:grapplerulrich@gmail.com" target="_blank">grapplerulrich@gmail.com</a><br>To: <a href="mailto:theme-reviewers@lists.wordpress.org" target="_blank">theme-reviewers@lists.wordpress.org</a><br>
Subject: Re: [theme-reviewers] Custom Post Metas in the Theme<div><div><br><br><p dir="ltr">What is wrong with using meta boxes? I do realise that in some cases you would loose data but there are other cases where it can be used for theme specific design. </p>
<p dir="ltr">I suppose these cases would need to be an exception. </p>
<div>On 22 Nov 2013 10:03, "Konstantin Kovshenin" <<a href="mailto:kovshenin@gmail.com" target="_blank">kovshenin@gmail.com</a>> wrote:<br><blockquote style="border-left:1px #ccc solid;padding-left:1ex">
<div dir="ltr">I think we should be looking for add_meta_box in Theme-Check and register_post_type for that matter.<div><br><br><div>On Fri, Nov 22, 2013 at 11:58 AM, Rohit Tripathi <span dir="ltr"><<a href="mailto:rohitink@live.com" target="_blank">rohitink@live.com</a>></span> wrote:<br>
<blockquote style="border-left:1px #ccc solid;padding-left:1ex">
<div><div dir="ltr"><div><span style="font-size:12pt">I recently approved a theme: Anew. But, later on the ticket was opened, and it was pointed out the theme is heavily dependant on Custom Post Metas for Implementation of Post Formats. I Obviously understand, that why its unacceptable. But, i didn't look in the Add New Post section of the theme. So, I clearly missed it.</span></div>
<div><span style="font-size:12pt"><br></span></div><div>Another Theme from the Same Author, which is currently a Featured Theme: <a href="http://wordpress.org/themes/hueman/" target="_blank">http://wordpress.org/themes/hueman/</a> also depends a lot on Custom Post Meta for implemention Post Formats, violating the WPORG's Functionality Guidelines. Those issues need to be resolved, as users who switch away from the theme would loose significant amount of data.</div>
<div><br></div><div>I am just writing this up here, as we reviewers should start looking carefully in the "Add New Post" areas, as we generally do not look there. </div><div><br></div><div>:)</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>Konstantin
</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>_______________________________________________
theme-reviewers mailing list
<a href="mailto:theme-reviewers@lists.wordpress.org" target="_blank">theme-reviewers@lists.wordpress.org</a>
<a href="http://lists.wordpress.org/mailman/listinfo/theme-reviewers" target="_blank">http://lists.wordpress.org/mailman/listinfo/theme-reviewers</a></div></div></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></div></div><span><font color="#888888"><span style="color:rgb(153,153,153)">Towfiq I.</span><br>
</font></span></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">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>