janwagner/jwd-minimal-wp — pull request #1 by grappler (2013-05-30 19:31:47+00:00)
Title: Improve header.php and add translation

Hi, I saw your tweet and had a look at your theme. I like your idea.

Here are a couple of ways you could improve it.

* You can style or make changes to the title in the functions.php like done [here](https://github.com/grappler/jwd-minimal-wp/blob/master/functions.php#L36-L56)
* Updated the favicon links to the new best practice and also to `stylesheet_directory` so that they can be updated to when using a child theme.
* The WordPress generator is not needed. WordPress creates this on its own.
* [This](https://github.com/grappler/jwd-minimal-wp/blob/master/functions.php#L14) allows rss theme support.
* The styles should be enqueued in the `functions.php`
* This is the same for scripts. I would not load jQuery unless you need it. It is an extra file to load. If it is required then it can be added to the enqueue for the JavaScript file as a dependency. [More Here](http://codex.wordpress.org/Function_Reference/wp_enqueue_script)
* I also added translation support and add the `.pot` file.

Let me know if you have any questions.