http://themeshaper.com/forums/ Help In Shaping WordPress Themes en Wed, 22 May 2013 05:14:02 +0000 http://themeshaper.com/forums/topic/98-978-dev-release-breaks-layout#post-25840 Mon, 12 Mar 2012 00:41:15 +0000 nimmolo 25840@http://themeshaper.com/forums/ <p>thanks Helga!!! </p> http://themeshaper.com/forums/topic/98-978-dev-release-breaks-layout#post-25838 Mon, 12 Mar 2012 00:23:56 +0000 helgatheviking 25838@http://themeshaper.com/forums/ <p>lean and mean. </p> <p>one last thing: you could put that image in your theme folder if you wanted (instead of hard linking it)</p> <pre><code>src=&quot;&lt;?php echo stylesheet_directory_uri() . &#39;/images/WP9-winter.png&#39;;?&gt;&quot;</code></pre> <p>also, looks like you can mark this as resolved </p> http://themeshaper.com/forums/topic/98-978-dev-release-breaks-layout#post-25837 Sun, 11 Mar 2012 23:59:46 +0000 nimmolo 25837@http://themeshaper.com/forums/ <p>OK. got it. Sorry for deleting the bloated code, did it before i read your reply.</p> <p>Here it is on a diet. I didn't know that about not having to add the action but of course it makes sense, it's built into the conditional syntax in header-extensions.php.</p> <p>Anything else I could pare out of here?</p> <p>I started with the childtheme_override_access() because the mobile childtheme does indeed override thematic_access() and i wanted them to be similar. Unnecessary here though.</p> <pre><code>// Let&#39;s add those mobile navmenus function register_mobile_navmenus() { if (function_exists( &#39;register_nav_menus&#39; )) { register_nav_menus( array( &#39;primary-menu&#39; =&gt; __( &#39;Masthead Navigation&#39;, &#39;thematic&#39; ), &#39;mobile-menu&#39; =&gt; __( &#39;Mobile Navigation&#39;, &#39;thematic&#39;)) ); } } add_action(&#39;init&#39;, &#39;register_mobile_navmenus&#39;); // HEADER ACTIONS, Nimmo style. function remove_defaults() { remove_action(&#39;thematic_header&#39;,&#39;thematic_brandingopen&#39;,1); remove_action(&#39;thematic_header&#39;,&#39;thematic_access&#39;,9); } add_action(&#39;init&#39;,&#39;remove_defaults&#39;); add_action(&#39;thematic_header&#39;,&#39;thematic_access&#39;,1); add_action(&#39;thematic_header&#39;,&#39;thematic_brandingopen&#39;,2); // Open #branding and add the branding image this way // In the header div function childtheme_override_brandingopen() { echo &#39;&lt;div id=&quot;branding&quot;&gt;&lt;a href=&quot;&#39; . home_url() . &#39;&quot;&gt;&lt;img src=&quot;/mistersaturdaynight/wp-content/uploads/header-images/WP9-winter.png&quot;&gt;&lt;/a&gt;&#39;; } // Dis the blog title // In the header div function childtheme_override_blogtitle() { } // Dis the blog description // In the header div function childtheme_override_blogdescription() { } // End of functions that hook into thematic_header()</code></pre> http://themeshaper.com/forums/topic/98-978-dev-release-breaks-layout#post-25836 Sun, 11 Mar 2012 23:49:40 +0000 helgatheviking 25836@http://themeshaper.com/forums/ <p>i think you have some code cleanup you could stand to do to make the code more elegant and therefore easier to maintain in the future:</p> <p>you don't need to do childtheme_override_something() and add_action('action', 'childtheme_override_something');</p> <p>if you are using overrides, you don't need to remove_action() the original action</p> <p>for sanity's sake you shouldn't name functions childtheme_override if they don't actually override anything, like your childtheme_override_init_navmenu. there isn't a thematic_init_navmenu function. it works, sure, but it gets confusing. </p> <p>why are you overriding access? doesn't look like you are doing anything except moving it? remove_action, then add_action back on new hook. </p> http://themeshaper.com/forums/topic/98-978-dev-release-breaks-layout#post-25835 Sun, 11 Mar 2012 23:29:06 +0000 nimmolo 25835@http://themeshaper.com/forums/ <p>OK no that didn't do it. I'm reading helga's post below now, i will make those changes...</p> <p>It's my inelegant code and the double branding close.</p> <p>For some reason, under 9.7.8 the new childtheme_override_brandingclose() now ran IN ADDITION TO thematic_branding_close(), not INSTEAD OF.</p> <p>So I'm now removing thematic_branding_close() manually instead of relying on the childtheme function override. Didn't use to have to do that!!!</p> <p>I think that could be a problem in header-extensions.php of 9.7.8, i will now investigate. </p> http://themeshaper.com/forums/topic/98-978-dev-release-breaks-layout#post-25834 Sun, 11 Mar 2012 23:11:40 +0000 nimmolo 25834@http://themeshaper.com/forums/ <p>OK - *edit* see below :) </p> http://themeshaper.com/forums/topic/98-978-dev-release-breaks-layout#post-25833 Sun, 11 Mar 2012 23:06:24 +0000 nimmolo 25833@http://themeshaper.com/forums/ <p>Och! My mistakes are foiled by smart people once again.</p> <p>Thanks!</p> <p>Jeez louise you would think i would have thought to check the dev tools but I was SO SURE my markup was a-ok. </p> http://themeshaper.com/forums/topic/98-978-dev-release-breaks-layout#post-25832 Sun, 11 Mar 2012 22:31:06 +0000 helgatheviking 25832@http://themeshaper.com/forums/ <p>cause i'm ninja like that. your explanation is more thorough however </p> http://themeshaper.com/forums/topic/98-978-dev-release-breaks-layout#post-25831 Sun, 11 Mar 2012 21:48:31 +0000 middlesister 25831@http://themeshaper.com/forums/ <p>Like you say, the #wrapper is no longer wrapping the page but only the header. Of course #main will not be affected by wrappers max-width since it is no longer inside it.</p> <p>This means that there is an extra closing div tag somewhere in the header. A quick look at your source code reveals that it has to do with the #branding. It is closed twice.</p> <p>Don't know how your code looks like, but there is a thematic_brandingopen as well as a thematic_brandingclose so make sure you don't close it twice by adding the closing div tag in thematic_brandingopen. I honestly don't know if this has to do with upgrading to the latest dev release. Both of these filters were there before so it's probable this issue has been there all along but just not visible until now.</p> <p>EDIT - hah Helga was before me :-) </p> http://themeshaper.com/forums/topic/98-978-dev-release-breaks-layout#post-25830 Sun, 11 Mar 2012 21:44:52 +0000 helgatheviking 25830@http://themeshaper.com/forums/ <p>the problem is not w/ thematic. you have some improperly closed html markup somewhere.</p> <p>in chrome dev tools your site looks like:<br /> <pre><code>&lt;div id=&quot;wrapper&quot;&gt; &lt;div id=&quot;header&quot;&gt;&lt;/div&gt; &lt;/div&gt; &lt;div id=&quot;main&quot;&gt; &lt;/div&gt;</code></pre> http://themeshaper.com/forums/topic/98-978-dev-release-breaks-layout#post-25828 Sun, 11 Mar 2012 19:59:20 +0000 nimmolo 25828@http://themeshaper.com/forums/ <p>After reading enough encouragements I installed the latest dev release from Thematic4You.</p> <p>Suddenly #main does not inherit anything from #wrapper?!?!?</p> <p>My layout is not behaving properly at all - #wrapper seems to stop after the header, and #main is following its own rules; it's not obeying wrapper's max-width and is using 100% of screen width. Check it at andrewnimmo.com/mistersaturdaynight if you like.</p> <p>There were no problems before I updated the Thematic and no changes since, either.</p> <p>Obviously I will revert in a few days but I'm posting this because it might be of interest to developers. </p>