theme review feedback fixes

2013-08-10 18:41:27 +09:00 · 2013-08-10 18:41:27 +09:00 · a64527376c
commit a64527376c
parent 58bda6c606
22 changed files with 23 additions and 25 deletions
--- a/404.php
+++ b/404.php
@ -2,7 +2,7 @@
 /**
 * The template for displaying 404 pages (Not Found).
 *
- * @package _s
+ * @package wp386
 */

 get_header(); ?>
--- a/archive.php
+++ b/archive.php
@ -4,7 +4,7 @@
 *
 * Learn more: http://codex.wordpress.org/Template_Hierarchy
 *
- * @package _s
+ * @package wp386
 */

 get_header(); ?>
--- a/comments.php
+++ b/comments.php
@ -7,7 +7,7 @@
 * handled by a callback to wp386_comment() which is
 * located in the inc/template-tags.php file.
 *
- * @package _s
+ * @package wp386
 */

 /*
--- a/content-page.php
+++ b/content-page.php
@ -2,7 +2,7 @@
 /**
 * The template used for displaying page content in page.php
 *
- * @package _s
+ * @package wp386
 */
 ?>

--- a/content-single.php
+++ b/content-single.php
@ -1,6 +1,6 @@
 <?php
 /**
- * @package _s
+ * @package wp386
 */
 ?>

--- a/content.php
+++ b/content.php
@ -1,6 +1,6 @@
 <?php
 /**
- * @package _s
+ * @package wp386
 */
 ?>

--- a/footer.php
+++ b/footer.php
@ -4,7 +4,7 @@
 *
 * Contains the closing of the id=main div and all content after
 *
- * @package _s
+ * @package wp386
 */
 ?>

--- a/header.php
+++ b/header.php
@ -4,7 +4,7 @@
 *
 * Displays all of the <head> section and everything up till <div id="main">
 *
- * @package _s
+ * @package wp386
 */
 ?><!DOCTYPE html>
 <html <?php language_attributes(); ?>>
@ -42,9 +42,7 @@
 				<div class="sidebar-affix">
 					<div class="sidebar-outer">
 						<div class="sidebar-container">
-							<?php if ( ! dynamic_sidebar( 'sidebar-1' ) ) : ?>
-
-							<?php endif; ?>
+							<?php dynamic_sidebar( 'sidebar-1' ); ?>
 						</div>
 					</div>
 				</div>
--- a/image.php
+++ b/image.php
@ -2,7 +2,7 @@
 /**
 * The template for displaying image attachments.
 *
- * @package _s
+ * @package wp386
 */

 get_header();
--- a/inc/customizer.php
+++ b/inc/customizer.php
@ -2,7 +2,7 @@
 /**
 * _s Theme Customizer
 *
- * @package _s
+ * @package wp386
 */

 /**
--- a/inc/extras.php
+++ b/inc/extras.php
@ -4,7 +4,7 @@
 *
 * Eventually, some of the functionality here could be replaced by core features
 *
- * @package _s
+ * @package wp386
 */

 /**
--- a/inc/jetpack.php
+++ b/inc/jetpack.php
@ -3,7 +3,7 @@
 * Jetpack Compatibility File
 * See: http://jetpack.me/
 *
- * @package _s
+ * @package wp386
 */

 /**
--- a/inc/template-tags.php
+++ b/inc/template-tags.php
@ -4,7 +4,7 @@
 *
 * Eventually, some of the functionality here could be replaced by core features
 *
- * @package _s
+ * @package wp386
 */

 if ( ! function_exists( 'wp386_content_nav' ) ) :
--- a/inc/wpcom.php
+++ b/inc/wpcom.php
@ -2,7 +2,7 @@
 /**
 * WordPress.com-specific functions and definitions
 *
- * @package _s
+ * @package wp386
 */

 global $themecolors;
--- a/index.php
+++ b/index.php
@ -8,7 +8,7 @@
 * E.g., it puts together the home page when no home.php file exists.
 * Learn more: http://codex.wordpress.org/Template_Hierarchy
 *
- * @package _s
+ * @package wp386
 */

 get_header(); ?>
--- a/no-results.php
+++ b/no-results.php
@ -4,7 +4,7 @@
 *
 * Learn more: http://codex.wordpress.org/Template_Hierarchy
 *
- * @package _s
+ * @package wp386
 */
 ?>

--- a/page.php
+++ b/page.php
@ -7,7 +7,7 @@
 * and that other 'pages' on your WordPress site will use a
 * different template.
 *
- * @package _s
+ * @package wp386
 */

 get_header(); ?>
--- a/sass/style.scss
+++ b/sass/style.scss
@ -241,7 +241,7 @@ object {

 /* Global styles */

-%border-box {
+.border-box {
  -webkit-box-sizing: border-box;
  -moz-box-sizing: border-box;
  box-sizing: border-box;
--- a/search.php
+++ b/search.php
@ -2,7 +2,7 @@
 /**
 * The template for displaying Search Results pages.
 *
- * @package _s
+ * @package wp386
 */

 get_header(); ?>
--- a/searchform.php
+++ b/searchform.php
@ -2,7 +2,7 @@
 /**
 * The template for displaying search forms in _s
 *
- * @package _s
+ * @package wp386
 */
 ?>
 <form role="search" method="get" class="search-form" action="<?php echo esc_url( home_url( '/' ) ); ?>">
--- a/sidebar.php
+++ b/sidebar.php
@ -2,7 +2,7 @@
 /**
 * The Sidebar containing the main widget areas.
 *
- * @package _s
+ * @package wp386
 */
 ?>
 	<div id="secondary" class="widget-area" role="complementary">
--- a/single.php
+++ b/single.php
@ -2,7 +2,7 @@
 /**
 * The Template for displaying all single posts.
 *
- * @package _s
+ * @package wp386
 */

 get_header(); ?>