yogiben / meteor-admin

A complete admin dashboard solution
https://atmospherejs.com/yogiben/admin
GNU General Public License v3.0
827 stars 261 forks source link

FR: editable table #35

Open dcsan opened 9 years ago

dcsan commented 9 years ago

it would be great if the table view allowed editing there's an atmosphere package for this http://handsontable.com/

yogiben commented 9 years ago

Completely agree.

WillHall commented 9 years ago

This is awesome. Let's talk logistics. The more features we stitch onto the the tableviews the more complex things are going to get (of course).

The functionality of editing a table cell, I don't think would be too difficult - but for each cell type we would need a callback. As well as provide a method to users to define their custom edit callbacks.

Perhaps instead of cell, we think of it at the row level. Though, I think instead of rolling another dependency in we could pretty easily write one specific to admin.

row.on edit

  1. create model of row
  2. create and populate text fields in the cells of the row

row.on save

  1. validate changes against model a. publish model if changed
  2. destroy row model

rinse. repeat.

yogiben commented 9 years ago

Yes. Before all this, I need to implemented nested value support e.g. 'profile.name'.

WillHall commented 9 years ago

If the datatables PR is feasible it already does nested values.