views blocks list shows two scrollbars [#1077300]

Comment	File	Size	Author
#13	views_blocks_list_shows_two_scrollbars-1077300-13.patch	1.72 KB	fxarte
#8	context.context_ui_blocklist-1077300-#5.patch	1.48 KB	fxarte
#5	context.context_ui_blocklist-1077300-#5.patch	1.48 KB	fxarte
#1	1077300.context.context-ui-views-list.patch	410 bytes	joachim
	context-blocks-views-scrollbars.png	60.3 KB	joachim

Comment #1

joachim commented 8 March 2011 at 12:08

Status:

Active

» Needs review

Status	File	Size
new	1077300.context.context-ui-views-list.patch	410 bytes

Here's a patch.

It just removes a max-height. Is there a purpose to this that I'm not seeing?

Log in or register to post comments

Comment #2

hefox commented 23 October 2011 at 04:10

Version:	6.x-3.x-dev	» 7.x-3.x-dev
Category:	bug	» feature
Status:	Needs review	» Needs work

If a single module provides a huge amount of blocks it will make finding blocks by other modules as keep having to scroll past alll these blocks by that module.

That ui really sucks for huge amount of blocks as is, this would make it worse. Not sure of a better ui.

Log in or register to post comments

Comment #3

joachim commented 23 October 2011 at 07:37

One scrollable inside another is pretty unusable UI -- something that's hardly ever seen on the web I think. It means your scroll wheel does unpredictable things. Moreover, having one scrollbar right next to the other makes it hard to aim for one of them.

How about collapsible fieldsets?

Log in or register to post comments

Comment #4

hefox commented 23 October 2011 at 16:45

The ui of features/context ui in general is :(.

How about how views does it for fields/arguments/etc? Module drop down at top that filters the list :O! Worked well for views, imo.

Log in or register to post comments

Comment #5

fxarte commented 4 November 2011 at 02:01

Status	File	Size
new	context.context_ui_blocklist-1077300-#5.patch	1.48 KB

Some UI improvement, although not exactly views UI dropdown type, but using collapsible field-sets. It includes the patch from #1

Log in or register to post comments

Comment #6

fxarte commented 9 November 2011 at 18:56

Status:

Needs work

» Needs review

Log in or register to post comments

Comment #7

hefox commented 9 November 2011 at 19:37

Your patch is 404ing o.O

Log in or register to post comments

Comment #8

fxarte commented 11 November 2011 at 18:26

Status	File	Size
new	context.context_ui_blocklist-1077300-#5.patch	1.48 KB

Let's see now.

There seems to be a bug, or url encoding missing on drupal.org file upload functionality. Because added the "#" character the file link is not being interpreted completely, if you replace the hash with "%23", you'll get the file.
Also, I will avoid using the hash character in the file names in the future.
this is the link:
http://drupal.org/files/context.context_ui_blocklist-1077300-%235.patch

Log in or register to post comments

Comment #9

fxarte commented 21 November 2011 at 20:12

Status:

Needs review

» Fixed

Log in or register to post comments

Comment #10

hefox commented 21 November 2011 at 20:39

why list this as fixed?

Log in or register to post comments

Comment #11

mraichelson commented 22 November 2011 at 04:19

Just to tack another note onto this one, based on how OS X Lion does it's scrollbar behaviors by default (show no scrollbars, only show them when scrolling) this interface becomes a very difficult thing to try to use on Lion.

Log in or register to post comments

Comment #12

fxarte commented 22 November 2011 at 19:53

Status:

Fixed

» Needs review

Hi hefox, it is not fixed, I just did it to reassign the ticket back to you. I could not find another way to notifying you.
I should've added this comment right away, my bad.

Log in or register to post comments

Comment #13

fxarte commented 1 February 2012 at 19:08

Status	File	Size
new	views_blocks_list_shows_two_scrollbars-1077300-13.patch	1.72 KB

New version of patch #8, with some UI improvements:

Excluding patch from #1.
Adding vertical-align:top to #context-blockform td.selector.
Removing height style declaration on: #context-blockform .context-blockform-selector to allow the block list grow automatically.

Log in or register to post comments

Comment #14

wbobeirne commented 15 February 2012 at 18:04

Status:

Needs review

» Reviewed & tested by the community

#13 is a solid patch, the only drawback is the lack of ctrl+f to find a block because all the fields are collapsed, but that's a small price for the improved readability.

Log in or register to post comments

Comment #15

bfr commented 30 May 2012 at 06:02

My alternative:
http://drupal.org/node/1463834 (patch for context)
http://drupal.org/project/context_accordion (separate helper module if you dont want to patch)

The thinking is similar but uses JQuery UI accordion instead. I Also tried to put some thinking to the CSS styling instead of just changing the technical side of the form elements.

Context accordion