Clean up the CSS for Block module

Tagging.

Tagging!

This is harder than I suspected...
It's not rly clear what belongs to the base.css.
I tried something.

This needs to be called somewhere:
Here all all the links to block.css

modules\block\block.admin.inc
      12    drupal_add_css(drupal_get_path('module', 'block') . '/block.css');
      80    drupal_add_css(drupal_get_path('module', 'block') . '/block.css');

do we need to call both files?

Aad I just tried to disable *every* css line and it still looks good in stark. So we don't need the base file here.

Ok ow I understand what we are trying to do :).
Attached patch with new approach, the theme specific stuff needs to be looked at carefully.

Hmmm actually we don't need to load the css anymore in the region page.

+++ b/themes/seven/style.cssundefined
@@ -994,3 +994,38 @@
+#blocks tr.region-title td {
+  font-weight: bold;
+}
+#blocks tr.region-message {
+  font-weight: normal;
+  color: #999;
+}
+.block-region {
+  background-color: #ff6;
+  margin-top: 4px;
+  margin-bottom: 4px;
+  padding: 3px;

I'm not sure we need to rip this out of core block and place it in every theme.

This leads to duplication. I do think we should move the block demo styling as it looks bad in stark. But maybe we can leave some basic styling. I don't know...

22 days to next Drupal core point release.

I feel like we should use the same approach as #1216976: Clean up the CSS for Comment module when patching this module.

Styles — especially the administrative styles in block.css — need to live in this module's directory, not duplicated amongst the core themes (and any active theme wanting to use block demonstration mode). It seems appropriate to relocate all of these styles to block.admin.css.

This patch renames block.css to block.admin.css, and fixes drupal_add_css() calls within block.admin.inc. I tested against Bartik, Garland, and Stark block admin/demo pages.

Other than splitting into block.admin.theme.css and block.admin.base.css... :) This is great.

#11: 1216950-block-css-cleanup-11.patch queued for re-testing.

Adding current sprint tag :)

See #12 still rtbc, bot hickup

Committed to 8.x. Thanks.

Cleaning up tags/component.

Even more clean up for the latest build.

Create a new issue.