Commit graph

49 commits

Author SHA1 Message Date
Truong Giang
9fcb8dab40 Correct code indentation 2017-07-27 00:37:56 +07:00
Jasper de Groot
34ab8ea00f Remove superfluous check from comment template (#935) 2017-07-14 11:52:14 -04:00
Dmitry Mayorov
8e67439859 Add comment navigation template tags 2017-06-30 15:25:57 +05:00
David A. Kennedy
f56a070c2b Remove landmark roles
* With better browser support, these roles aren't needed now.
* Most newer browsers imply them via HTML5 semantics. See: http://html5doctor.com/on-html-belts-and-aria-braces/

Fixes #1132
2017-06-23 12:41:05 -04:00
jrfnl
fd701834ff Add translator comments as the strings contain variables. 2016-12-23 15:59:15 +01:00
jrfnl
8dbca77003 Fix translatability of comments title.
As per the guidelines in https://codex.wordpress.org/I18n_for_WordPress_Developers#Plurals

This fixes an error notice which WPCS will start throwing once the - soon to be released - 0.10.0 version has become stable.
2016-12-23 15:47:35 +01:00
Tammie Lister
e39f5e55db Merge pull request #1009 from B-07/patch-4
Add end comment
2016-12-23 12:20:57 +00:00
Bappi
354ff52901 Remove period from file header 2016-09-27 12:20:40 +06:00
Bappi
c7cea499fc Add end comment
On line no 36
2016-08-17 08:24:36 +06:00
jrfnl
8470f55f92 Avoid going in and out of PHP unnecessarily.
Closes #854

Includes minor alignment changes for easier readability/detection of start/end of conditions.
2015-11-11 22:59:40 +01:00
Grant Palin
af0a4749be Update comparisons to resolve the strict comparison warnings. See https://github.com/Automattic/_s/issues/784 2015-08-07 09:59:26 -07:00
PJ
de9c9888b7 Updated documentation in comments at the top of files.
I tried to standardize and expand the documentation and links to
codex.wordpress.org in the comments at the top of most files.
2015-07-12 12:38:18 -04:00
Fränk Klein
175524922c Fix Codesniffer errors. 2015-05-27 22:19:14 +02:00
Fränk Klein
2aef92e5a3 Fix remaining PHPCS errors:
* Add ignoring in template-tags.php and comments.php.
* Escape header text color in custom-header.php.
2015-05-11 09:18:36 +02:00
Fränk Klein
062105ac9d Escape translations on comments.php.
See #737.
2015-05-05 12:37:07 +02:00
Philip Arthur Moore
ff6337943b Better coding standards. See #737. 2015-04-29 08:32:11 +07:00
Stefan Brechbühl
e45aff27ed Consistent navigation
Handle the comment navigation the same way as the new the_post_navigation() and the_posts_navigation().
2015-02-14 12:21:53 +01:00
Jack Lenox
0b96c02e02 Undid my multiline function corrections 2014-06-25 10:31:51 +01:00
Jack Lenox
776eaf6e94 Correcting some irregular capitalisation and coding standards issues 2014-06-25 00:13:36 +01:00
Satrya
c780103027 There is no more _s_comment() in inc/template-tags.php 2014-02-07 23:08:48 +07:00
obenland
28a3044da9 _s: Minor formatting update. 2014-01-20 20:34:10 -08:00
Ulrich Pogson
4f2b0fab63 Add necessary arguments 2013-12-24 00:26:50 +01:00
Ulrich Pogson
a29f2773aa Remove arguments 2013-12-22 22:01:16 +01:00
Ulrich Pogson
cb52da6f23 Remove _s_comment 2013-12-22 21:33:16 +01:00
Philip Arthur Moore
6264437ff9 Add braces around single-line conditionals. See http://make.wordpress.org/core/2013/11/13/proposed-coding-standards-change-always-require-braces/. 2013-11-14 19:49:50 +07:00
Aahan Krish
1e1fd565e1 Removed extra indentation on div#comments 2013-10-10 22:46:32 +05:30
Brad Parbs
bf6ecae7c4 minor code comment comma grammar fixes 2013-08-21 12:04:29 -05:00
Patrick Rauland
3d98e794d3 also changing overload to override in comments.php & archive.php 2013-08-20 07:27:46 -05:00
michaeldcain
31331b4b64 Switch "navigation-" back to "-navigation"
Original edit not needed for `[class*="navigation"]` selector. Matches
other class/id navigation naming conventions.
2013-07-11 16:42:45 -04:00
Jason Hoffmann
b790c82151 Comments.php Spelling Issue 2013-05-20 14:12:09 -04:00
obenland
456cd3dd01 _s: Removes unnecessary switch in and out of PHP. 2013-04-28 04:13:55 -07:00
obenland
6056c1ed66 _s: Re-introduce nav- prefix for previous/next links.
This way we can select both links with `[class*="nav-"]`.
2013-04-05 09:34:08 -07:00
Philip Arthur Moore
fe19f5433b Ditch all @since DocBlocks from theme. 2013-03-18 14:53:23 -05:00
obenland
1ba2f62752 _s: Use .screen-reader-text rather than .assistive-text.
With 3.6 introducing many HTML5 improvements the need for custom
searchform and comment templates will vanish. To cope with core's
.screen-reader-text class, let's make the switch early.

Commit also adds core's focus styles for screen reader texts, so they
can actually be used.
2013-03-13 15:39:05 -07:00
obenland
c4f2aa32ce _s: Formatting changes:
- Reduce element comments to either the element id or the first element
class.
- Tabs > Spaces.
- Improve adherence to formatting guidelines.
2013-03-04 16:41:38 -08:00
obenland
ad57a4af29 _s: Move role declaration to the end of the element for consistency. 2013-02-27 14:51:13 -08:00
obenland
fa3d86936a _s: Simplify and unify navigation classes throughout Underscores.
For more information about the philosophy behind the new approach,
see http://24ways.org/2012/a-harder-working-class/
2013-02-27 14:47:03 -08:00
obenland
d99f4c8115 _s: Adhere to WordPress CSS class naming conventions. See
http://make.wordpress.org/core/handbook/coding-standards/css/#selectors
for details.
2013-02-27 14:03:03 -08:00
Brian Fegter
ec4c855158 Updated 'comments callback' location comment
The function has been moved outside of 'functions.php' and into 'inc/template-tags.php'. This directs developers to the proper location for the callback.
2013-02-12 10:40:31 -06:00
obenland
6793718e8a _s: Provide a context for the comments title strings.
There are languages that have more than one plural form. Sometimes these
languages need to add the %1$s placeholder in the original singular
version to help differentiate between these plural forms.

In case the original singular version ('One thought on
“%2$s”' in this case) is used elsewhere, the translation
with the faulty %1$s will be used, possibly ending up with the
placeholder not being replaced.

The context will prevent that from happening by seperating the strings.
2013-01-14 16:38:39 -08:00
Lance Willett
473b835299 Merge pull request #25 from kwight/comments-php-comment
Corrected reference to location of _s_comment(). Props kwight.
2012-08-06 16:14:24 -07:00
Kirk Wight
603369e307 Corrected reference to location of _s_comment()
* _s_comment() is now located in inc/template-tags.php, rather than
functions.php as stated
2012-06-12 17:42:14 -04:00
kobenland
b9fa5c0dd2 Add missing 'class' comments 2012-05-23 21:01:41 +02:00
Philip Arthur Moore
55ab0fa5d6 _s: In comments.php there's a line that reads "if comments are closed and no comments are...". Let's get rid of the "no" since comments must be present in order for the .nocomments text to be shown.
git-svn-id: https://wpcom-themes.svn.automattic.com/_s/@9560 d957f892-c61d-0410-b221-f235e6eecf30
2012-05-15 16:51:49 -05:00
Michael Fields
c82269d831 _s: Remove password-protected message from comments.php. fixes #1217
git-svn-id: https://wpcom-themes.svn.automattic.com/_s/@9480 d957f892-c61d-0410-b221-f235e6eecf30
2012-04-24 14:38:15 -05:00
Ian Stewart
22522ac0c4 _s: adding instructions on forking _s to the readme and cleaning up a php comment in comments.php that gets missed in the find and replace pattern
git-svn-id: https://wpcom-themes.svn.automattic.com/_s/@8893 d957f892-c61d-0410-b221-f235e6eecf30
2012-02-08 13:54:26 +00:00
Ian Stewart
0a42cc02cb _s: Adding ARIA navigation roles to the all the nav elements
git-svn-id: https://wpcom-themes.svn.automattic.com/_s/@8851 d957f892-c61d-0410-b221-f235e6eecf30
2012-02-03 14:22:08 +00:00
Ian Stewart
139dda72f5 _s: cleaner, more consistent ids and classes for comments.php
git-svn-id: https://wpcom-themes.svn.automattic.com/_s/@8833 d957f892-c61d-0410-b221-f235e6eecf30
2012-02-01 19:35:10 +00:00
Ian Stewart
edfeccc255 _s: initial fork of Toolbox 2012-01-07 03:25:13 +00:00