The region.tpl will use ".region-header" for the header region's wrapper div. Unfortunately, admin/build/block also uses ".region-header" for drag-and-drop table rows.

Which means if you set a width on one of your region.tpl's wrapper div, you'll also apply that same with to the table row which can screw up the layout of the blocks admin page.

CommentFileSizeAuthor
#4 zen-prevent-block-admin-mistyling-686014-4.patch759 bytesbarraponto

Comments

johnalbin’s picture

johnalbin
they/他
Primary language English
Location Taipei, Taiwan
commented
Priority: Critical » Normal

Its hardly critical since it doesn't break anything. It just causes an odd appearance on the blocks admin page.

Drupal 7's block admin page was re-written to allow D7's region.tpl to use the .region-header/etc classes.

jix_’s picture

jix_ commented

Just write div.region-[name] :)

shruti.sheth’s picture

shruti.sheth commented
Component: CSS/HTML Markup » layout.css

1.You can apply css to your block admin page only* as follows

.page-admin-build-block #page-wrapper #header #block-header div.region-header {
// Your css code
}

2. This will have affect to only your block admin page and none other pages.

barraponto’s picture

barraponto
he / him
commented
Status: Active » Needs review
StatusFileSize
new zen-prevent-block-admin-mistyling-686014-4.patch759 bytes

#3 is hardly the case, but #2 is a proper solution to a problem we introduce in pages.css, since Zen guides the developer to use .header-region as a selector. Patch fixes that.

johnalbin’s picture

johnalbin
they/他
Primary language English
Location Taipei, Taiwan
commented
Status: Needs review » Needs work
johnalbin’s picture

johnalbin
they/他
Primary language English
Location Taipei, Taiwan
commented
Issue summary: View changes
Status: Needs work » Closed (won't fix)