cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Chris Suich" <>
Subject Review Request 14285: Add multi-row action support to UI list views (CLOUDSTACK-1579)
Date Mon, 23 Sep 2013 19:48:43 GMT

This is an automatically generated e-mail. To reply, visit:

Review request for cloudstack.

Repository: cloudstack-git



I have added the logic and hooks for allowing list views to indicate they support multiselect
rows and added hooks for list view actions to indicate they are a multiselect action. The
data for the selected rows is stored and passed to callbacks in the same 'context' variable
as with non-multiselect callbacks, so implementing multiselect should remain consistent.

This is purely a functional update and there is cosmetic work to be done shortly by someone
other than myself (likely Brian Federle).

Some notes for those interested:
- $.remove() vs $.detach(): remove will delete the DOM element while detach simply removes
it from view, allowing it to be added later and re-used
- List views can indicate they support multi-select with 'multiSelect: true'
- Actions can indicate they are a multi-select action with 'isMultiSelectAction: true' and
'isHeader: true'
- This patch also includes a new list view flag called 'noActionCol' which intentional hides
the 'Quickview' on rows which can be handy when the rows become too wide and look weird
- Currently, when one or more multi-select rows are selected, non-multi-select actions (such
as Add) are hidden and replaced with the multi-select actions, however the idea was proposed
to add another row of buttons below the non-multi-select buttons when this is the case - this
is still open for discussion as there was no consensus


  ui/plugins/netapp PRE-CREATION 
  ui/scripts/ui/widgets/dataTable.js 66d7095 
  ui/scripts/ui/widgets/listView.js 076b3ab 



Testing has been done in conjunction with a UI plugin I am developing. Additional testing
will be performed as this is integrated into existing parts of the UI.


Chris Suich

  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message