Hi!

I made a small module to improve the block selection system, screenshot:
Context Accordion screenshot

However, i think it would be better to integrate to Context UI itself.
I'll provide a quick patch if you are interested.
Project page:
http://drupal.org/project/context_accordion

CommentFileSizeAuthor
#2 context-integrate_context_accordion_to_context_ui-1463834-2.patch7.37 KBbfr
#1 context-integrate_context_accordion_to_context_ui-1463834-1.patch7.15 KBbfr
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

bfr’s picture

bfr CreditAttribution: bfr commented
Assigned: bfr » Unassigned
Status: Active » Needs review
FileSize
context-integrate_context_accordion_to_context_ui-1463834-1.patch7.15 KB

Here's a patch. It's done with format-patch like they now should be, so if you will use it, please apply with git am filename.patch as described here:
http://drupal.org/node/1146430

edit: forgot to mention that i also fixed some CSS coding standard issues in the context_ui.css file.

bfr’s picture

bfr CreditAttribution: bfr commented
FileSize
context-integrate_context_accordion_to_context_ui-1463834-2.patch7.37 KB

Minor misthinking in the CSS. Apply as described before.

bfr’s picture

bfr CreditAttribution: bfr commented
Issue summary: View changes

Image is suddenly broken, trying to fix it.

bfr’s picture

bfr CreditAttribution: bfr commented
Issue summary: View changes

Last edit did not do the trick.

bfr’s picture

bfr CreditAttribution: bfr commented
Issue summary: View changes

Still not working

bfr’s picture

bfr CreditAttribution: bfr commented

This does not seem to get too much interest. Should i close it?

For alternative take on this problem, see this thread:
http://drupal.org/node/1077300

bfr’s picture

bfr CreditAttribution: bfr commented
Issue summary: View changes

Still not working

askibinski’s picture

askibinski CreditAttribution: askibinski commented

+1 for context_accordion integration. Major usability improvement!

That tiny scroll area has been bothering me for ages.
I know you can fix it with CSS but in my opinion it should work with themes like Seven out of the box.

Hope a maintainer can look at the patch.

pbfleetwood’s picture

pbfleetwood CreditAttribution: pbfleetwood commented

I haven't tried the patch--I have way too many patches, as is--but the Context Accordion module is brilliant. It makes a huge improvement, a sorely needed improvement.

Here's a big +1 for integrating it with this module.

itaine’s picture

itaine CreditAttribution: itaine commented

"This is LOVE! Simply outstanding!"

Have way too many modules as is... +1 for commitment :-)

yannickoo’s picture

yannickoo
Location Berlin
CreditAttribution: yannickoo commented
Status: Needs review » Needs work

First all tabs and whitespaces should be removed.

+++ b/context_ui/context_ui.cssundefined
@@ -89,85 +87,120 @@ table.context-admin input.form-text { width:90%; }
+/** ¶

Whitespace

+++ b/context_ui/context_ui.cssundefined
@@ -89,85 +87,120 @@ table.context-admin input.form-text { width:90%; }
+	display: block;

Tabs...

tekante’s picture

tekante CreditAttribution: tekante commented

This appears to break the inline editing functionality of context_ui. Can anyone state whether that functionality works for them when using this patch?

tekante’s picture

tekante CreditAttribution: tekante commented
Issue summary: View changes

Typo