Hi, first of all thanks for the great module. Right now I am using it for creating an ubercart 'product admin' (which ubercart doesn't really have!) with purposes to let people somehow have a dashboard alike function for modifying the products in their store.

Since personally I have experience in the hassle of editing 1000+ products to ubercart that will take you lots of time and bandwidth, this module will save the time and bandwidth for ubercart user.

The only glitch is, with table layout its seem that lots of viewing space is wasted and 'wont fit to monitor' display experiences. So if the display output can be changed to div. I believe the display output can be formatted better due to div flexibility thus creating a better display layout for user.

Anyway, thanks for the great module. it really helps ease the process of editing thousand of ubercart products.

duckz

CommentFileSizeAuthor
#3 editview-list-mode.patch.txt490 bytesfriolator
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

friolator’s picture

I'd also like to know if this is possible. as it is right now, this module is useless to us because the generated table is about 3x wider than my widescreen monitor, and obviously can't fit our page design.

this would really be a welcome addition to the module.

friolator’s picture

Last night I messed around with this a little bit, and was able to get it to work. It's not a proper patch, nor is it well tested, but it does seem to do the trick.

File: editview.theme.inc, starting at line 100:

<?php>
  return theme('table', $header, array($row), array('class' => 'editview-row')) . $buttons;
?>

Change to:

<?php>
  return theme('item_list', $row) . $buttons;
?>

The result is an unordered list, with each form item inside an <li> tag.

In editview.css, there's some css that hides the labels for the form items, you'll need to override this accordingly in your own CSS:

.editview-field .form-item label,
.editview-field .form-item .description,
.editview-field fieldset legend {
  display: none;
}

We just changed all of these to display: inline since that works for our purposes.

Again, not heavily tested but it does seem to do the trick. It'd be great to have the option to choose table or list modes for form display.

friolator’s picture

FileSize
490 bytes

Here's a patch that does the above:

--- editview.theme.inc	2009-05-04 09:28:02.000000000 -0400
+++ editview.theme-1.inc	2009-08-10 09:54:54.000000000 -0400
@@ -96,7 +96,7 @@ function theme_editview_node_form($form)
   $handler = $view->style_plugin;
   $active = !empty($handler->active) ? $handler->active : '';
   $order = !empty($handler->order) ? $handler->order : 'asc';
-  return theme('table', $header, array($row), array('class' => 'editview-row')) . $buttons;
+  return theme('item_list', $row) . $buttons;
 }
 
 /**
Agileware’s picture

Version: 6.x-1.0-beta1 » 6.x-1.x-dev

The table layout will have to remain as you don't want to go backwards and lose existing functionality.

An option will have to be presented to the user when they select to use an editview view that allows them to choose which format to use.
Then we need different templates for each format.

thomas.frobieter’s picture

Could you please implement this feature in the next version?
In my case display style "table" is not very useful, I fixed it only by using CSS (display:list-style;) this works, but it's not very clean.

Would be great to be able to switch between list-style, table-style and maybe unformatted also.

//Edit: Oh and.. thank you very, very much for the module at all! It's soooo useful.

With kind regards,
Thomas

MBroberg’s picture

See issue at http://drupal.org/node/581488#comment-4703120 for new functionality!