THEME: HDboilerplate - 1.1

[hxf148]

HDboilerplate - 1.1

A HTML5, Widget ready clean template.

Theme URL - ​http://hdbolierplate.com
Author URL - ​http://haroldfudge.com

SVN - ​http://themes.svn.wordpress.org/hdboilerplate/1.1
ZIP - ​http://wordpress.org/extend/themes/download/hdboilerplate.1.1.zip?nostats=1

Diff with previous version: http://themes.trac.wordpress.org/changeset?old_path=/hdboilerplate/1.0&new_path=/hdboilerplate/1.1

All previous tickets for this theme: http://themes.trac.wordpress.org/query?col=id&col=summary&col=keywords&col=owner&col=status&col=resolution&keywords=~theme-hdboilerplate&order=id

[emiluzelac]

• Theme URL - ​http://hdbolierplate.com does not exist, domain name is not even registered yet
• Images such as ArvhokICIAIKrNL.jpg should be within a /images/ folder
• ​license.txt is empty
• Theme looks based on Starkers, please note that you must carry on their license ​https://github.com/nathanstaines/starkers-html5/blob/master/LICENSE.md as well as either leave or add the copyright reference to this Theme. GPL license requires that credits are left or if not you will need to add them in your Theme.
• ​/languages/ folder contains Starkers reference.
• Style embedding is bad coding practice

<span style="color:#c0c0c0;">
<div style="margin-bottom:8px;">

etc, please remove styles similar to this and add them in your stylesheet. When you embed styles they will be impossible to overwrite.

Remove completely

//error_reporting(E_ALL);
//ini_set('display_errors', '1');

this is also not how debug should be anyways, you can simply open your wp-config.php and change wp_debug to true

/**
 * For developers: WordPress debugging mode.
 *
 * Change this to true to enable the display of notices during development.
 * It is strongly recommended that plugin and theme developers use WP_DEBUG
 * in their development environments.
 */
define('WP_DEBUG', true);

JS should not be used like this

<script src="<?php echo get_template_directory_uri() ?>/jquery-1.7.2.min.js" type="text/javascript"></script>
<script type="text/javascript">
function mainmenu(){
$(" #nav a").removeAttr("title");
$(" #nav ul ").css({display: "none"}); // Opera Fix
$(" #nav li").hover(function(){
		$(this).find('ul:first').css({visibility: "visible",display: "none"}).show(0);
		},function(){
		$(this).find('ul:first').css({visibility: "hidden"});
		});
}
$(document).ready(function()
{
	mainmenu();
});
</script>

please add the functions into example.js and call them from functions.php also there's no need to add jQuery, WordPress? handles this pretty nicely.

---

There's more than enough items that must be corrected before next submission. If you have any questions please ask them here, you can also visit ​Theme Review for more details about this review.