Sidebar is a bad name - it encourages the perpetuation of the stupid 3 column CMS paradigm, by making people think of block regions as "side bars". Shame it was committed with that name. andre drumm wrote:
Issue status update for http://drupal.org/node/18382
Project: Drupal Version: cvs Component: block.module Category: tasks Priority: normal Assigned to: drumm Reported by: drumm Updated by: drumm -Status: fixed +Status: patch Attachment: http://drupal.org/files/issues/block.module_8.diff (769 bytes)
Yep, there was a typo. Good call on changing 'region' to 'sidebar'.
drumm
Previous comments: ------------------------------------------------------------------------
March 4, 2005 - 12:29 : drumm
Attachment: http://drupal.org/files/issues/block.module_6.diff (3.44 KB)
I got tired of hunting through the block admin page for the block I wanted so I went and changed so it is organized by left region, right region, and disabled. Looks a lot like the permissions admin page, especially since it uses the same styles. And there were two copies of this $row = array(...); chunk, which was a bit weird. So that gets removed too.
------------------------------------------------------------------------
March 4, 2005 - 12:46 : drumm
Oh yeah.. throttle can disappear. And that other patch I made slipped in. Here is a better one.
------------------------------------------------------------------------
March 4, 2005 - 12:46 : drumm
Attachment: http://drupal.org/files/issues/block.module_7.diff (3.39 KB)
------------------------------------------------------------------------
March 5, 2005 - 03:09 : Dries
Committed to HEAD.
------------------------------------------------------------------------
March 5, 2005 - 03:48 : Mad Maks
typo: Right sidabar