Just putting this here to see if there is any interest. I *might* be able to fund some of this development.

Idea would be that anyplace you see a "+" or edit icon in Storm in would use the Popups API to bring up a proper Ajaxy window to add/edit a node inline.

See http://drupal.org/project/popups and in particular this sub module that acts on CCK Node Reference fields http://drupal.org/project/popups_reference

Comments

dbt102’s picture

dbt102 commented

I viewed the popups screencast and it seems like a cool feature. Could it be hooked in such a way that it was a storm option, meaning you could turn it on if you want it?

Magnity’s picture

Magnity commented

What needs to change in Storm for this to work? I'm fine with supporting it as long as it doesn't affect those who don't install it at all.

cglusky’s picture

cglusky commented

I think it would end up being a submodule similar to http://drupal.org/project/popups_reference so I don't think anything would need to change in Storm. I see it as a way to possibly get a bit better user experience, but I could be wrong.

I like Storm and where you are headed but I needed to have my site up last week with a multi-organizational/team set-up. And if I use OG/CCK/Views/Node Reference/Popups API plus about ten more modules and a few hacks I can get close in the short term. Unfortunately, I do not have the time/funding to spare right now for Storm development.

I will continue to watch Storm and this issue. And I would be happy to talk anytime about how my experiment goes with my site and how Storm might support the use case in the future. I am sure there will be much head banging and many lessons learned.

Thanks,
Coby

homoludens’s picture

homoludens commented
Status: Active » Needs review

for the quick fix:
i have made popups "on + sign" working with changing 2 lines of code - using popups attribute:
in file storm.module change function storm_icon_add to this:

function storm_icon_add($path, $item, $params=array()) {
  global $user;
  $type = $item->type;
  // Work around during 436922
  if ($type=="storminvoiceitem") {
    $type = "storminvoice";
  } // End workaround
  $af = $type .'_access';
  if (!$af('create', $item, $user)) return '';
  $attributes = array('class' => 'popups-form');
  return storm_icon_l('application_add', $path, t('Add'), '', $params, $attributes);
}

basically only adding line $attributes = array('class' => 'popups-form'); and adding $attributes in next line.

it is working for me. it is only important to go to theme settings and set content area selector ( probable div#content or similar)

good thing with this is that popup api is not mandatory dependency it only adds one more class to "+" sign. thou, you need to leave checked "Scan all pages for popup links."

If all goes well I can, maybe, implement it little better with all changes that should make this nice feature.

Magnity’s picture

Magnity commented
Status: Needs review » Fixed

Committed the change to add, edit and delete links.

Status: Fixed » Closed (fixed)

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