[drupal-devel] [task] Organize block admin page by region
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: patch 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. drumm
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: patch Oh yeah.. throttle can disappear. And that other patch I made slipped in. Here is a better one. 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.
On Friday 04 March 2005 13:46, drumm wrote:
Oh yeah.. throttle can disappear. And that other patch I made slipped in. Here is a better one.
I haven't tested the patch(es) yet, but assuming they work as advertised, a big +1 for the concept. Great idea! Scott -- -----------------------+------------------------------------------------------ Scott Courtney | "I don't mind Microsoft making money. I mind them scott@4th.com | having a bad operating system." -- Linus Torvalds http://4th.com/ | ("The Rebel Code," NY Times, 21 February 1999) | PGP Public Key at http://4th.com/keys/scott.pubkey
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: patch Attachment: http://drupal.org/files/issues/block.module_7.diff (3.39 KB) 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.
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
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
Op zondag 6 maart 2005 04:07, schreef Andre Molnar:
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".
+1 on regions instead of "sidebars". Bèr -- [ Bèr Kessels | Drupal services www.webschuur.com ]
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
+1 on regions instead of "sidebars".
+1. -- Morbus Iff ( whooooooo's hoooouuuuuse? ) Technical: http://www.oreillynet.com/pub/au/779 Culture: http://www.disobey.com/ and http://www.gamegrene.com/ icq: 2927491 / aim: akaMorbus / yahoo: morbus_iff / jabber.org: morbus
Morbus Iff wrote:
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
+1 on regions instead of "sidebars".
+1.
Drupal uses the 3 column paradigm. As such, 'sidebar' is a lot less confusing. How about calling it 'column' which seems to have a bit of both? -- Dries Buytaert :: http://www.buytaert.net/
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
+1 on regions instead of "sidebars".
+1.
Drupal uses the 3 column paradigm. As such, 'sidebar' is a lot less confusing. How about calling it 'column' which seems to have a bit of
I still like region better. The downside of "column" is that we'd never be able to happily and semantically do "wide" columns, like a header or footer (what the hell is a "header column"?). Likewise, it's difficult to imagine a column within a column, or a sidebar within a sidebar, but quite easy to imagine a region within a region. "Sidebar" may be less confusing in the short run, but naming parts of Drupal solely per the paradigm (ie. renaming "taxonomy" "categories", which still bums me out) is bad. Remember: the greatest honor of a paradigm is to be broken. -- Morbus Iff ( dare you overpower my stench of eeeevil? ) Technical: http://www.oreillynet.com/pub/au/779 Culture: http://www.disobey.com/ and http://www.gamegrene.com/ icq: 2927491 / aim: akaMorbus / yahoo: morbus_iff / jabber.org: morbus
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
+1 on regions instead of "sidebars".
+1.
Drupal uses the 3 column paradigm. As such, 'sidebar' is a lot less confusing. How about calling it 'column' which seems to have a bit of
I still like region better. The downside of "column" is that we'd never be able to happily and semantically do "wide" columns, like a header or footer (what the hell is a "header column"?). Likewise, it's difficult to imagine a column within a column, or a sidebar within a sidebar, but quite easy to imagine a region within a region. "Sidebar" may be less confusing in the short run, but naming parts of Drupal solely per the paradigm (ie. renaming "taxonomy" "categories", which still bums me out) is bad. Remember: the greatest honor of a paradigm is to be broken.
I agree with Dries in that the naming should reflect the current behaviour not some envisoned future feature set. Now blocks can only be 'left' or 'right', ie. in sidebars. I even think that 'columns' is worse then 'sidebars', since 'columns' relate to tables, while 'sidebars' cite the meaning of what we are talking about. If a patch is done and accepted to allow true regions, then it is more than fine to rename 'sidebars' to 'regions'. But until then, sidebar is more than suffice. But then the column header of the blocks table should also be renamed 'sidebar' and not 'region'. Drupal developers were not afraid of providing new names for old stuff, just see the 'access control' menu in the 4.6RC, or how bundles where renamed to categories in 4.5. BTW I was also against renaming taxonomy to categories in the menu, just for the record, but that does not matter now :) Goba
On Sun, 6 Mar 2005, Gabor Hojtsy wrote:
I agree with Dries in that the naming should reflect the current behaviour not some envisoned future feature set. Now blocks can only be 'left' or 'right', ie. in sidebars.
No, that depends on the theme. It is perfectly ok to put one "sidebar" on top or at the bottom. Cheers, Gerhard
I agree with Dries in that the naming should reflect the current behaviour not some envisoned future feature set. Now blocks can only be 'left' or 'right', ie. in sidebars.
No, that depends on the theme. It is perfectly ok to put one "sidebar" on top or at the bottom.
Yes, but it will still be called 'right' and 'left' in the block admin interface, and that is the point. Goba
Gabor Hojtsy wrote:
No, that depends on the theme. It is perfectly ok to put one "sidebar" on top or at the bottom. Indeed. You can put both of them wherever you like, it's only the admin interface and class and ID attributes that refer to 'sidebar', 'right' and 'left'.
This isn't some future functionality, it's available right now, and these terms are already out of date. +1 on Region
Yes, but it will still be called 'right' and 'left' in the block admin interface, and that is the point. All that shows is that we need to rethink the block admin too :) Block-region-1, Block-region-2?
It's going to be more confusing isn't it - we're back to the usual tension between making it powerful and flexible vs making it easy for novice admins... I too mourn the loss of taxonomy, I just think the 'categories' reference is so shallow that it adds confusion. -- adrinux (aka Adrian Simmons) <http://adrinux.perlucida.com> e-mail <mailto:adrinux@gmail.com> AOL/Yahoo IM: perlucida, Microsoft: adrian@perlucida.com
No, that depends on the theme. It is perfectly ok to put one "sidebar" on top or at the bottom.
Indeed. You can put both of them wherever you like, it's only the admin interface and class and ID attributes that refer to 'sidebar', 'right' and 'left'.
This isn't some future functionality, it's available right now, and these terms are already out of date.
+1 on Region
Yes, but it will still be called 'right' and 'left' in the block admin interface, and that is the point.
All that shows is that we need to rethink the block admin too :) Block-region-1, Block-region-2?
It's going to be more confusing isn't it - we're back to the usual tension between making it powerful and flexible vs making it easy for novice admins...
Unless someone comes up with some names in place of 'left' and 'right', this discussion is going nowhere. Goba
Adrian wrote:
All that shows is that we need to rethink the block admin too :) Block-region-1, Block-region-2?
It's going to be more confusing isn't it - we're back to the usual tension between making it powerful and flexible vs making it easy for novice admins...
Gabor Hojtsy wrote:
Unless someone comes up with some names in place of 'left' and 'right', this discussion is going nowhere.
Perhaps we need to come up with some mechanism for the theme to name those regions, and have that name show up in the block admin page. Thus, most existing themes would in fact name them "left" and "right" but some other theme could name them "region 1" and "region 2." At a minimum we need to consider that eventually the block admin page is going to have to deal with blocks which will never end up on the "left" or "right" of the page, and thus using those terms will remain confusing. -- Chris Johnson
Unless someone comes up with some names in place of 'left' and 'right', this discussion is going nowhere.
Perhaps we need to come up with some mechanism for the theme to name those regions, and have that name show up in the block admin page. Thus, most existing themes would in fact name them "left" and "right" but some other theme could name them "region 1" and "region 2."
At a minimum we need to consider that eventually the block admin page is going to have to deal with blocks which will never end up on the "left" or "right" of the page, and thus using those terms will remain confusing.
We are talking about a short term solution now. 4.6 will go out with 'sidebar' as grouping titles, 'region' as column name and 'left' and 'right' as column values, unless this is changed. The code now contains these. 4.6 is not going to have a mechanism for themes to specify the regions. We are looking for a good short term solution. Goba
Gabor Hojtsy wrote:
4.6 is not going to have a mechanism for themes to specify the regions. We are looking for a good short term solution
Right on. If future versions of Drupal support more than 3 columns, I'll be the first to change 'sidebar' to 'region'. -- Dries Buytaert :: http://www.buytaert.net/
Gabor Hojtsy wrote:
We are talking about a short term solution now. 4.6 will go out with 'sidebar' as grouping titles, 'region' as column name and 'left' and 'right' as column values, unless this is changed. The code now contains these. 4.6 is not going to have a mechanism for themes to specify the regions. We are looking for a good short term solution.
Goba
For 4.6, I suggest we be consistent and just call them left or right sidebars. Sorry that I did not make it clear I was talking about "the future" beyond 4.6. -- Chris Johnson
Perhaps we need to come up with some mechanism for the theme to name those regions, and have that name show up in the block admin page. Thus, most existing themes would in fact name them "left" and "right" but some other theme could name them "region 1" and "region 2."
At a minimum we need to consider that eventually the block admin page is going to have to deal with blocks which will never end up on the "left" or "right" of the page, and thus using those terms will remain confusing.
The big roadblock for custom theme regions is having multiple themes per site. This is a valid use case (sections.module, or even a 'mobile' theme for PDA/Cell phones). I'd suggest you check out some earlier discussions on this in the archives. Steven Wittens
Op maandag 7 maart 2005 00:11, schreef Steven Wittens:
The big roadblock for custom theme regions is having multiple themes per site. This is a valid use case (sections.module, or even a 'mobile' theme for PDA/Cell phones). I'd suggest you check out some earlier discussions on this in the archives.
This indeed is a problem. But can we not tackle it right here and now? My proposal: 1) a theme registers all its regions by returning an array from a called hook_regions. These should be choosed from a documented lists of "suggested good defaults". 2) the engine caches these, in a smart way: private (only in theme X), globals (in more than one theme), leftovers (only in one theme, not the same as private). 3) The UI presents these regions, overlapping is solved. (eg box_grey gives "header" and so does friendseletric, we should have only one "header"). The UI has a flag to define a *default* region. 4) The regions are now iterated as follows: 4.1) Global regions are filled by engine and cached per role and engine. (like now) 4.2) Private regions are filled by theme and cached per role and theme 4.3) Leftovers are appended to the default region and cached per role and theme 5) theme_page($content, $region='all') collects all these strings either from cache or from a rebuild, and returns them as HTML. $region can be 'all' (same as now) or $region ='none' only content area is returned (for popups, and performance!) or a string (I can see some nice uses for this, but ATM not yet used) ($region=header, to print only the header area) or $region ='none'. The naming has been discussed on drupal.org a lot already. I think, that if we agree on this issue Steven raises, we have more then enough said and documented to make up a nice program of demands for this module. Only thing is that we then need to find someone with ten fingers (or less) and a brain with some PHP and Drupal code in it, to actually make it. Hell, I might even volunteer for this, but I have to ask my client first. :) Regards, Bèr -- [ Bèr Kessels | Drupal services www.webschuur.com ]
Dries Buytaert wrote:
Morbus Iff wrote:
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
+1 on regions instead of "sidebars".
+1.
Drupal uses the 3 column paradigm. As such, 'sidebar' is a lot less confusing. How about calling it 'column' which seems to have a bit of both?
Drupal.org uses the 3 column paradigm. Drupal does not have to. andre
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: stefan nagtegaal -Status: fixed +Status: patch Please after typo corrections, update also the related *.po files which are in the contributions... Setting status to patch again untill this gets fixed.. stefan nagtegaal Previous comments: ------------------------------------------------------------------------ March 4, 2005 - 19: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 - 19:46 : drumm Oh yeah.. throttle can disappear. And that other patch I made slipped in. Here is a better one. ------------------------------------------------------------------------ March 4, 2005 - 19:46 : drumm Attachment: http://drupal.org/files/issues/block.module_7.diff (3.39 KB) ------------------------------------------------------------------------ March 5, 2005 - 10:09 : Dries Committed to HEAD. ------------------------------------------------------------------------ March 5, 2005 - 10:48 : Mad Maks typo: Right sidabar ------------------------------------------------------------------------ March 5, 2005 - 22:20 : drumm Attachment: http://drupal.org/files/issues/block.module_8.diff (769 bytes) Yep, there was a typo. Good call on changing 'region' to 'sidebar'. ------------------------------------------------------------------------ March 6, 2005 - 00:00 : Dries Committed to HEAD.
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: Goba Status: patch Stefan, there is no reason to update PO files one by one for every single string change. The 4.6.0 RC is going to have more string changes, possibly more string changes than one a day. It is not feasible to update the po files all the time. Goba Previous comments: ------------------------------------------------------------------------ March 4, 2005 - 19: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 - 19:46 : drumm Oh yeah.. throttle can disappear. And that other patch I made slipped in. Here is a better one. ------------------------------------------------------------------------ March 4, 2005 - 19:46 : drumm Attachment: http://drupal.org/files/issues/block.module_7.diff (3.39 KB) ------------------------------------------------------------------------ March 5, 2005 - 10:09 : Dries Committed to HEAD. ------------------------------------------------------------------------ March 5, 2005 - 10:48 : Mad Maks typo: Right sidabar ------------------------------------------------------------------------ March 5, 2005 - 22:20 : drumm Attachment: http://drupal.org/files/issues/block.module_8.diff (769 bytes) Yep, there was a typo. Good call on changing 'region' to 'sidebar'. ------------------------------------------------------------------------ March 6, 2005 - 00:00 : Dries Committed to HEAD. ------------------------------------------------------------------------ March 6, 2005 - 20:58 : stefan nagtegaal Please after typo corrections, update also the related *.po files which are in the contributions... Setting status to patch again untill this gets fixed..
participants (13)
-
Adrian Simmons -
Andre Molnar -
Bèr Kessels -
Chris Johnson -
Dries Buytaert -
drumm -
Gabor Hojtsy -
Gerhard Killesreiter -
Goba -
Morbus Iff -
Scott Courtney -
stefan nagtegaal -
Steven Wittens