Perhaps I wasn't clear with my earlier reply ...<br><br>I agree with Mike on the title should be empty if the author chooses not to write one; although I also would not recommend this particular item fall under the Theme Review guidelines. The end-users will make it clear if they think the Theme author's design concept fits their needs ... or not.<br>
<br>I'm still not thrilled with the method I am using in my themes but this has given me a few more ideas to play with.<br><br><div class="gmail_quote">On Tue, Mar 8, 2011 at 3:45 PM, 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;">I would use:<div><br></div><blockquote style="margin: 0pt 0pt 0pt 40px; border: medium none; padding: 0px;">
<div>if ( ! get_the_title() )</div></blockquote><div><br></div><div>Rather than:</div><div class="im"><div><br>
</div><blockquote style="margin: 0pt 0pt 0pt 40px; border: medium none; padding: 0px;"><div>$title = the_title( '', '', false );<br>if ( empty( $title ) )</div></blockquote><div><br></div>
</div><div>But, that doesn't really address your underlying point, which I believe is valid, but outside the scope of the Theme Review guidelines. I think we've got a good balance, by requiring a permalink to the Single-Post view, but not dictating *how* that permalink appears. Our concern is the UX; the design consideration can remain a matter between the developer and the end users. I'd prefer we, as Theme reviewers, not get in the middle of that.</div>
<div><br></div><div><font color="#888888">Chip</font><div><div></div><div class="h5"><br><br><div class="gmail_quote">On Tue, Mar 8, 2011 at 2:39 PM, <span dir="ltr"><<a href="mailto:michael@mfields.org" target="_blank">michael@mfields.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;">
Right on, I can totally see that need. The main thing the erks me about<br>
solutions such as this is that they take the authors intentions and throw<br>
them out the window. If I am writing a post and I choose to leave the<br>
title blank, I would expect the title to be blank. I do not want the title<br>
replaced with "no title". To me it has the effect of a designer leaving in<br>
FPO images in a finished design ... IMO it looks sloppy.<br>
<br>
If a permalink is needed, I would suggest that it be added somewhere else<br>
in the template in a conditional check. the_title can still be used for<br>
this:<br>
<br>
$title = the_title( '', '', false );<br>
if ( empty( $title ) ) {<br>
print '<a href="' . esc_url( get_permalink() ) . '">Permalink</a>';<br>
}<br>
<br>
-Mike<br>
<div><div></div><div><br>
<br>
> The issue arises when Post Title is the only available permalink to the<br>
> Single-Post view. In this case, the developer may want to provide text<br>
> such<br>
> as "No Title", etc., so that the permalink still gets displayed.<br>
><br>
> Chip<br>
><br>
> On Tue, Mar 8, 2011 at 2:15 PM, <<a href="mailto:michael@mfields.org" target="_blank">michael@mfields.org</a>> wrote:<br>
><br>
>> Is there a need for a conditional test?<br>
>><br>
>> I wrap all my code within the function and the conditional as pect is<br>
>> handled for you:<br>
>><br>
>> the_title( '<h2>', '</h2>' );<br>
>><br>
>> This way the tags will not be printed if the title is empty.<br>
>><br>
>> -Mike<br>
>><br>
>> > Ran across this suggestion on the WPORG support forums:<br>
>> ><br>
>> ><br>
>> <a href="http://wordpress.org/support/topic/handling-display-of-title-when-none-exists?replies=1" target="_blank">http://wordpress.org/support/topic/handling-display-of-title-when-none-exists?replies=1</a><br>
>> ><br>
>> > I like this approach; it is cleaner than performing a conditional test<br>
>> in<br>
>> > the template file.<br>
>> ><br>
>> > Just thought I'd pass it along...<br>
>> ><br>
>> > Chip<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>
>><br>
>><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>
> _______________________________________________<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>
<br>
<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>
</div></div></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>