bueltge / wordpress-admin-style

This plugin reference elements from wp-admin in an overview with the necessary markup and CSS classes to help you to develop WordPress compliant.
http://wpengineer.com/2226/new-plugin-to-style-your-plugin-on-wordpress-admin-with-default-styles/
GNU General Public License v2.0
960 stars 124 forks source link

Two Column Markup, sidebar H3 element outside postbox DIV #8

Closed userabuser closed 11 years ago

userabuser commented 11 years ago

Hey Frank...

The copy and paste markup for the two column layout needs a slight adjustment, currently the sidebar has no H3 element styling for its background. Snippet shown below with accompanying screenshot.

https://github.com/bueltge/WordPress-Admin-Style/blob/master/wp-admin-style.php#L286

The following markup;

            <div class="meta-box-sortables">
                <h3><span>Sidebar Content Header</span></h3>
                <div class="postbox">
                    Content space
                </div> <!-- .postbox -->
            </div> <!-- .meta-box-sortables -->

...produces: whoops

Where as;

            <div class="meta-box-sortables">
                <div class="postbox">
                    <h3><span>Sidebar Content Header</span></h3>
                    Content space
                </div> <!-- .postbox -->
            </div> <!-- .meta-box-sortables -->

...produces: better

userabuser commented 11 years ago

...actually it's also missing an inner DIV element that wraps the content

<div class="inner">
    content space
</div>
bueltge commented 11 years ago

Thanks for the hints, I have fixed the error to use it easier.