Comments on: Issues In the Sandbox http://en.dahnielson.com/2006/08/issues-in-the-sandbox.html Misleading the public since 1979 Sat, 13 Mar 2010 02:21:34 +0000 http://wordpress.org/?v=2.7.1 hourly 1 By: Anders Dahnielson http://en.dahnielson.com/2006/08/issues-in-the-sandbox.html/comment-page-1#comment-157 Anders Dahnielson Tue, 15 Aug 2006 18:59:27 +0000 http://en.dahnielson.com/2006/08/issues-in-the-sandbox.html#comment-157 Ok, I see about the quoting. But there's actually another issue fixed by the patch: in <code>index.php</code> line 19 there's a lot of \t\t\t\t in a single quoted string that hence need to be double quoted. That's what started it. From there I just went beserck and fixed all strings not knowing about the WordPress coding standards. Sorry. Ok, I see about the quoting. But there’s actually another issue fixed by the patch: in index.php line 19 there’s a lot of \t\t\t\t in a single quoted string that hence need to be double quoted.

That’s what started it. From there I just went beserck and fixed all strings not knowing about the WordPress coding standards. Sorry.

]]> By: Andy Skelton http://en.dahnielson.com/2006/08/issues-in-the-sandbox.html/comment-page-1#comment-156 Andy Skelton Tue, 15 Aug 2006 18:15:45 +0000 http://en.dahnielson.com/2006/08/issues-in-the-sandbox.html#comment-156 Hi Anders, and thanks for giving so much attention to our little theme. Some responses: Attribute quoting is valid with single- or double-quotes. They are interchangeable. Your patch actually makes the PHP code harder to read. See the <a href="http://codex.wordpress.org/WordPress_Coding_Standards" rel="nofollow">WordPress coding standards</a>. I have just re-tested R57 and found it working at intended. In fact it's live on WordPress.com right now and working fine. Thanks also for the suggestions. We have several revisions to go before we kick this bird out of the nest and I expect some of your ideas will be represented. Cheers! Hi Anders, and thanks for giving so much attention to our little theme. Some responses:

Attribute quoting is valid with single- or double-quotes. They are interchangeable. Your patch actually makes the PHP code harder to read. See the WordPress coding standards.

I have just re-tested R57 and found it working at intended. In fact it’s live on WordPress.com right now and working fine.

Thanks also for the suggestions. We have several revisions to go before we kick this bird out of the nest and I expect some of your ideas will be represented. Cheers!

]]>