Ah I see. So long as there's a permalink to the post, not necessary at the post title.<br>Thanks!<br><br><div class="gmail_quote">On Sun, Nov 21, 2010 at 8:40 AM, Chip Bennett <span dir="ltr"><<a href="mailto:chip@chipbennett.net">chip@chipbennett.net</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">No worries on posting this to theme-reviewers. That's what the mail-list is here for! :)<div>
<br></div><div>There are several ways to meet this requirement. One way is to provide some default text when no Post Title is defined:</div>
<div><br></div><div>if ( get_the_title() ) {</div><div> the_title();</div><div>} else {</div><div> echo "(No Title)"; // or whatever you want to echo in place of the Post Title</div><div>}</div><div><br>
</div><div>Another way is to add a permalink elsewhere in the Post, such as a "Permalink" link, or making the Post Date a permalink.</div><div><br></div><div>By the way, every Theme that has gone through the review process will have met this guideline. TwentyTen was approved prior to the current review process, and I cannot vouch for its adherence to the Theme Review Guidelines.</div>
<div><br></div><div>Chip<br><br><div class="gmail_quote"><div><div></div><div class="h5">On Sat, Nov 20, 2010 at 6:33 PM, Genkisan <span dir="ltr"><<a href="mailto:genkiemail@gmail.com" target="_blank">genkiemail@gmail.com</a>></span> wrote:<br>
</div></div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;"><div><div></div><div class="h5">
Hi<br><br>Sorry for posting this to the theme-reviewers email list as I'm unable to see the comment form at the trac ticket even after login.<br>Just
want to check regarding the "No Post Permalink is displayed when Post
Title is omitted", is it to do a check and if no title, display the
permalink? Something like<br><br>
if title == ''<br>the_permalink<br>else<br>the_title<br><br>Cos I'm unable to see how this is implemented in the twentyten or other approved themes.<br><br>Thanks<br><font color="#888888">Genkisan</font><div>
<div></div><div><br><br><div class="gmail_quote">On Sun, Nov 21, 2010 at 5:12 AM, WordPress Themes <span dir="ltr"><<a href="mailto:theme-reviewers@lists.wordpress.org" target="_blank">theme-reviewers@lists.wordpress.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;"><div>#1958: THEME: GoogleChrome - 1.3.2<br>
---------------------------+------------------------------------------------<br>
Reporter: genkisan | Owner: chipbennett<br>
</div> Type: theme | Status: closed<br>
Resolution: not-approved | Keywords: theme-googlechrome,<br>
---------------------------+------------------------------------------------<br>
Changes (by chipbennett):<br>
<br>
* status: assigned => closed<br>
* resolution: => not-approved<br>
<br>
<br>
Comment:<br>
<br>
'''Note: it appears that Theme Name is in violation of<br>
[<a href="https://chrome.google.com/extensions/intl/en/branding.html" target="_blank">https://chrome.google.com/extensions/intl/en/branding.html</a> Google's<br>
trademark policy with respect to "Google Chrome"]. Please provide evidence<br>
of permission granted for use of the "Google Chrome" name, or else re-<br>
submit Theme using a new name.'''<br>
<br>
'''Theme Check'''<br>
<br>
''Recommended''<br>
<br>
* Recommended: No reference to add_editor_style() was found in the theme.<br>
It is recommended that the theme implement editor styling, so as to make<br>
the editor content match the resulting post output in the theme, for a<br>
better user experience.<br>
* Recommended: No reference to add_custom_image_headerwas found in the<br>
theme. It is recommended that the theme implement this functionality if<br>
using an image for the header.<br>
* Recommended: No reference to add_custom_background() was found in the<br>
theme. If the theme uses background images or solid colors for the<br>
background, then it is recommended that the theme implement this<br>
functionality.<br>
* Recommended: No reference to post-formats was found in the theme. It is<br>
recommended that the theme implement this functionality.<br>
* Recommended: No reference to the_post_thumbnail() was found in the<br>
theme. It is recommended that the theme implement this functionality<br>
instead of using custom fields for thumbnails.<br>
* Recommended: No reference to post-thumbnails was found in the theme. If<br>
the theme has a thumbnail like functionality, it should be implemented<br>
with add_theme_support( \'post-thumbnails\' )in the functions.php file.<br>
* Recommended: could not find the file readme.txt in the theme.<br>
<br>
''Info''<br>
<br>
* Info: Possible hard-coded links were found in the file footer.php.<br>
* Line 7: <a href="<a href="http://ericulous.com/2008/09/09/wp-theme-google-" target="_blank">http://ericulous.com/2008/09/09/wp-theme-google-</a><br>
chrome" title="Sma<br>
* Info: At least one hard coded date was found in the file single.php.<br>
* Info: At least one hard coded date was found in the file index.php.<br>
<br>
'''Required'''<br>
* Code Quality<br>
* HTML: validation error on 404.php, due to same CSS IDs used in<br>
multiple search forms.<br>
* Theme Unit Check<br>
* General<br>
* Header Menu: breaks layout when many list items cause menu to break<br>
across rows<br>
* Post With No Title<br>
* No Post Permalink is displayed when Post Title is omitted<br>
<br>
'''Review Summary'''<br>
<br>
Theme cannot be accepted until items listed under '''Required''' are<br>
addressed. Once these issues have been resolved, please re-submit your<br>
Theme using the uploader tool on <a href="http://wordpress.org/extend/themes" target="_blank">wordpress.org/extend/themes</a>, and a new<br>
Trac ticket will be generated.<br>
<br>
Note:<br>
* Re-submitted Themes are given review priority over new Themes.<br>
* Prior to re-submitting, please ensure Theme meets all requirements in<br>
the [<a href="http://codex.wordpress.org/Theme_Review" target="_blank">http://codex.wordpress.org/Theme_Review</a> Theme Review Guidelines] and<br>
the [<a href="http://codex.wordpress.org/Theme_Unit_Test" target="_blank">http://codex.wordpress.org/Theme_Unit_Test</a> Theme Unit Tests].<br>
<br>
'''Responding To This Review'''<br>
<br>
Theme developer questions, comments, and feedback are welcome and<br>
encouraged. Please use one of the following methods:<br>
<br>
1) Leave a comment on this ticket<br>
2) Email the theme-reviewers email list<br>
3) Use the #wordpress-themes channel on <a href="http://irc.freenode.net" target="_blank">irc.freenode.net</a><br>
<font color="#888888"><br>
--<br>
Ticket URL: <<a href="http://themes.trac.wordpress.org/ticket/1958#comment:2" target="_blank">http://themes.trac.wordpress.org/ticket/1958#comment:2</a>><br>
</font><div><div></div><div>WordPress Themes <<a href="http://themes.trac.wordpress.org/" target="_blank">http://themes.trac.wordpress.org/</a>><br>
WordPress.org Theme Directory Reviews<br>
</div></div></blockquote></div><br>
</div></div><br></div></div>_______________________________________________<br>
theme-reviewers mailing list<div class="im"><br>
<a href="mailto:theme-reviewers@lists.wordpress.org" target="_blank">theme-reviewers@lists.wordpress.org</a><br>
</div><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">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>