Posted by Pasqualle on September 6, 2008 at 8:04pm
| Project: | Context |
| Version: | 6.x-2.0-beta7 |
| Component: | Code |
| Category: | feature request |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | closed (fixed) |
| Issue tags: | context-2.x-rc-blocker |
Issue Summary
version: 6.x
The forms should use theme templates.
Comments
#1
#2
I'm going to have to disagree with you here - the markup actually isn't complicated at all.
#3
theme template file for the admin landing page
additional new files:
context_ui/theme/context-ui-list-contexts.tpl.php
context_ui/help/getting-started.html
context_ui/css/contexts-list.css
#4
the missing one
#5
FYI, this is very interesting, I like some of the ideas (showing the summaries of the conditions/reactions for each context) while I miss some pieces of the old layout (tree indenting made for easy quick scanning).
I'd like to work on this more post 2.0 release -- for 2.1 perhaps?
#6
Do you have a good layout idea (mockup) for tree indenting? I would like to implement filters and sort also (like in views), so that could be problematic for the tree layout. But if someone knows a good layout, then please help me here..
#7
improved layout
2 new files moved to the main context_ui directory, to be able to generate one patch file..
#8
Looks good (visually).
+
+
<?php/*print $widgets;*/
?>
no thanks
<?php+ // list contexts (admin landing page)
?>
capitalize comment and adda period at the end
+<?php if (isset($close_div)): ?>+ </div>
+<?php endif; ?>
when's that used? it doesn't look like it's used anywhere
#9
$widgets should be the next patch which adds sort and filter
you are right about the comment
that $close_div part is closing the last "namespace>attribute" box. If the contexts array would be in a tree presentation, that template would be much readable with two foreach loops (div starting at foreach, div closing at foreach end).
foreach namespace-attribute<div>
foreach contexts in that namespace-atribute
//
endforeach
</div>
endforeach
It is possible to create the tree presentation of contexts in the template, but it would require going through the list twice.
#10
I like that, and you don't need to go through twice, just change
<?php+ $vars['contexts'][] = $item;
?>
to
<?php+ $vars['contexts']["$item->namespace-$item->attribute"][] = $item;
?>
#11
used:
$vars['contexts_tree'][$item->namespace][$item->attribute][$item->name] = $item;so it is now possible to easily create a bounding box for the namespace also..
and fixed all the problems from #8
#12
I've re-rolled this patch for the latest version of context, to enable people to keep working on this if they want.
There are a couple of issues that I can see in the code at a quick glance:
There's a require_once instead of a module_load_include.
The 'getting started' help page has no help in it, this will need to be addressed in a follow up patch.
#13
Patch attached.
#14
This should probably get into 2.x.
#15
Committed to CVS: http://drupal.org/cvs?commit=357362 Looks cool!
#16
Automatically closed -- issue fixed for 2 weeks with no activity.