[theme-reviewers] Friendly request? About themes comments section

[Justin Tadlock]

It works fine on single posts too (or anywhere else for that matter), not
just the home page.  I've used it in plugins.

I'm not sure if it's a flaw in the system though. comments_link(), from my
understanding, is only meant to link to the comments. If there are no
comments to link to, we probably shouldn't attempt to link to them.

With that said, I've long believed that the WP template tags could use an
overhaul. There are a lot of these little oddities that are confusing.

On Thursday, October 23, 2014, Philip M. Hofer (Frumph) <philip at frumph.net>
wrote:

>   That’s nice, but comment_popup_link() only works on the
> is_home/is_front_page while plugin authors and others who create unique
> loops that need to work elsewhere are apparently going to be out of luck.
>
> Say for example someone wants to put a comments link at the top of the
> post itself and the post is quite long, it will not work if there are no
> comments since #comments doesn’t exist.  It will not auto-realign to the
> comments area because comments_popup_link will not appear on the singles
>
> But hey, whatever you want.  Just pointing out a flaw in the system.
>
>
>  *From:* Justin Tadlock
> <javascript:_e(%7B%7D,'cvml','justin at justintadlock.com');>
> *Sent:* Thursday, October 23, 2014 8:37 AM
> *To:* Discussion list for WordPress theme reviewers.
> <javascript:_e(%7B%7D,'cvml','theme-reviewers at lists.wordpress.org');>
> *Subject:* Re: [theme-reviewers] Friendly request? About themes
> commentssection
>
>  If you go back and look at some of the original themes or even in the
> `wp-includes/theme-compat/comments.php`, you'll notice this is how it's
> done and has always been done as far as I can remember.  The `#comments`
> section isn't shown until there are actual comments.
>
> This is why I'd consider `comments_link()` and `get_comments_link()` the
> inappropriate functions when there are no comments for the post. You'd be
> linking to something that does not yet exist.
>
> I'd use `comments_popup_link()` instead, which will display a link to
> either `#comments` or `#respond` depending on the situation.  This function
> also takes into account things like whether comments are open, pings are
> open, or a password is required before showing the link.
>
>
> On Thu, Oct 23, 2014 at 5:47 AM, Philip M. Hofer (Frumph) <
> philip at frumph.net <javascript:_e(%7B%7D,'cvml','philip at frumph.net');>>
> wrote:
>
>> comments_link()   get_comments_link() appends #comments to the end of the
>> url
>>
>> If  the comments section displaying on the single pages/posts does not
>> have a id="comments" then the comments_link appending of the #comments is
>> worthless .. I see quite a few themes that only have the #respond section
>> but not the wrapper around it for #comments
>>
>> The most common reason is that dev's have it so that it's in the part of
>> the comments.php section that only shows up when the post actually has a
>> comment made for it ..
>>
>> ^ which it shouldn't, then it's not seen or used as a ref.  for the
>> #comments in the comments_link;   it should be the class for the wrapper
>> for the whole entire comments section - or at least shown as a marker
>> location regardless if the post has comments or not
>>
>> - Phil
>>
>>
>>
>>
>>
>>
>>
>> _______________________________________________
>> theme-reviewers mailing list
>> theme-reviewers at lists.wordpress.org
>> <javascript:_e(%7B%7D,'cvml','theme-reviewers at lists.wordpress.org');>
>> http://lists.wordpress.org/mailman/listinfo/theme-reviewers
>>
>
>
> ------------------------------
> _______________________________________________
> theme-reviewers mailing list
> theme-reviewers at lists.wordpress.org
> <javascript:_e(%7B%7D,'cvml','theme-reviewers at lists.wordpress.org');>
> http://lists.wordpress.org/mailman/listinfo/theme-reviewers
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.wordpress.org/pipermail/theme-reviewers/attachments/20141023/9268a380/attachment-0001.html>