Commit graph

36 commits

Author SHA1 Message Date
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