Shorten the block configuration page
marco.robotangel - December 18, 2007 - 23:47
| Project: | Drupal |
| Version: | 7.x-dev |
| Component: | block.module |
| Category: | task |
| Priority: | normal |
| Assigned: | andreashaugstrup |
| Status: | patch (code needs review) |
Description
Steps to reproduce: Try to configure a block.
The block configuration page is way too big. All those options are, especially for newbies, just overwhelming. The suggestion made at [1] is to shorten the page by collapsing 'role' and 'page specific visibility settings'. This would shorten the page a lot.
#1
tag for newbie
#2
I'm certainly a newbie. I have attached a patch that collapses the role visibility and page specific visibility fieldsets (setting collapsed to TRUE in the form definition in block.admin.inc).
#3
In the contrary to #202548, I'm not having major headaches with this. I'm just asking myself: Why do we collapse role and page visibility fieldsets and have the user specific visibility fieldset open. But I think it is okay this way.
#4
I assumed the user specific visibility would be used much more often than the other two.
#5
Applies cleanly. Code style is good.
We don't know this and have no easy way of finding out. Usability testing of this wouldn't give useful data because the result would be dependent on the task.
However my suspicion is that 'Page specific visibility settings' are the most commonly used. My opinion is that we should collapse 'User specific visibility settings', and move 'Page' first, leaving it collapsed. I think this is quite subjective though.
#6
collapse them all?
#7
collapsing them all is definitely edging on fieldsetitis -- wdyt?
#8
Collapsing all *looks* great on pages with custom blocks because they have a large textarea and the form just ends up looking like the node form. It looks a tiny bit more silly for blocks from modules because you are left with one textfield (the block title) and then three collapsed fieldsets.
Personally, I would collapse them all.
Collapsing all means having the submit button above the fold and that would remove the only annoyance I have when I configure blocks that only is to get a new title. It would help me when I'm only on the configure screen to check values (e.g. see what roles can see the block) because I only have to look through the 3 fieldset titles to find the info I need rather than browse over the entire form.
The trade off would be a more annoying block configuration screen when I have to use 2 or more of the fieldsets (setting roles *and* page visibility). In that case it would take marginally longer, but it's not something that would bother me a lot.
#9
I am going to set this to Drupal 7, but it should definitely get in. However it's a UI improvement, which is not allowed anymore in Drupal 6
#10
Patch attached that collapses all three fieldsets.
#11
What do others think about swapping the order to put 'page' first?
#12
It's impossible to say what people in general use the most. In my personal experience I would say that I use the "user specific" fieldset the least and I would enjoy to see that moved to the bottom. "role" and "page visibility" I use around the same amount and I have no opinion about which one should be the top-most fieldset.
#13
I use:
most - page
then - role
lastly - user
so +1 for page coming up the page.
#14
First I've seen of this issue but a couple of weeks ago I thought about submitting a patch to put 'page' first but never got round to doing it. So +1 here as well.