There are two main issues with this module for us.

  • one the structure is very ridged.
  • we don't want to show the left column in our case, some quick setting in the view should allow toggling of that display.

Patches on their way.......

CommentFileSizeAuthor
#6 move-to-tpl-and-toggle-view-options-1490410-5.patch17.38 KBroam2345
#5 move-to-tpl-and-toggle-view-options-1490410-4.patch17.38 KBroam2345
#4 move-to-tpl-and-toggle-view-options-1490410-3.patch15.36 KBroam2345
#3 move-to-tpl-and-toggle-view-options-1490410-2.patch15.36 KBroam2345
#2 move-to-tpl-and-toggle-view-options-1490410-1.patch12.74 KBroam2345
#1 move-to-tpl-and-toggle-view-options-1490410-0.patch9.08 KBroam2345

Comments

roam2345’s picture

roam2345 commented
StatusFileSize
new move-to-tpl-and-toggle-view-options-1490410-0.patch9.08 KB

First patch so far move to a template file that allows themers to quickly rearrange the output of the content.

roam2345’s picture

roam2345 commented
Status: Active » Needs review
StatusFileSize
new move-to-tpl-and-toggle-view-options-1490410-1.patch12.74 KB

Right there a tone of changes and fixes....

- moved to templates so its easy to theme this lot for themers.
- added option to allow web admin to hide the left column from views options_form.
- fixed the AJAX bug where request doesn't "know" anything about the function "node_form" because it doesn't include modules/node/node.pages.inc.

roam2345’s picture

roam2345 commented
StatusFileSize
new move-to-tpl-and-toggle-view-options-1490410-2.patch15.36 KB

Keep missing the new files on my patches sorry.

roam2345’s picture

roam2345 commented
StatusFileSize
new move-to-tpl-and-toggle-view-options-1490410-3.patch15.36 KB

Remove the AJAX callback when the left column is hidden.

roam2345’s picture

roam2345 commented
StatusFileSize
new move-to-tpl-and-toggle-view-options-1490410-4.patch17.38 KB

anther small issue

roam2345’s picture

roam2345 commented
StatusFileSize
new move-to-tpl-and-toggle-view-options-1490410-5.patch17.38 KB

and fix the css

rerooting’s picture

rerooting commented

for whatever reason, i can't get this patch to apply. no output whatsoever with git apply -v. anything i might be doing wrong? looking forward to helping to test this!

roam2345’s picture

roam2345 commented

@rerooting checkout the project from git first.

rerooting’s picture

rerooting commented

awesome! applied cleanly. ill let you know if i notice any issues.

rerooting’s picture

rerooting commented

works great! this was extremely helpful. we may be able to combine this with a patch we are working on to allow fields without unlimited cardinality.

mrfelton’s picture

mrfelton commented

@rerooting - I'd love to get hold of your patch for that one, as we need to be able to use for fields that only allow one value, where really radios should be shown and not checkboxes. How far far are you with that path? got anything you want us to test?

rerooting’s picture

rerooting commented

not much yet, we will let you know when we are ready to test. this is really for another issue entirely, just mentioning that this patch may offer a way to get around some of the complexities that has led the maintainer(s) to avoid non-unlimited cardinality with this widget.

roam2345’s picture

roam2345 commented

@rerooting could you let us know of the issue number that we can follow along with that patch? That cardinality issue was something I was about to tackle next :)

rerooting’s picture

rerooting commented

Sorry, we actually decided not to use entity reference view widget and instead used a patch for entityreference itself we found. We had wonderful results! For simple needs regarding exposing fields in option/radio lists and autocomplete, look no further than the patch we refer to on this comment!

http://drupal.org/node/1253776#comment-5813360

davidseth’s picture

davidseth commented

@rerooting - I have created a new module that allows for much more flexibility in creating the view. You can use the table view layout and add any addition fields you want. It also is much more light-weight in its implementation with less ajax calls back to Drupal.

It is currently in a sandbox: http://drupal.org/sandbox/davidseth/1489176. You will have to check out the 7.x-1.x branch.

rerooting’s picture

rerooting commented

Thanks david! We are doing fine with the patch to entityreference that I referred to, so we won't need any additional modules. I would highly suggest that all work along these lines focus on expanding this patch for entityreference.

bojanz’s picture

bojanz commented
Status: Needs review » Fixed

Committed #6.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.