search

DaltonPimmel / flexigrid

Automatically exported from code.google.com/p/flexigrid
0 stars 0 forks source link

Flexibility fix: add third parameter to cell process function pointing to flexgrid container #16

Open GoogleCodeExporter opened 8 years ago

GoogleCodeExporter commented 8 years ago 
Hello,

Bit from my column mode:

colModel : [
  {
    display: '',
    name : 'select',
    process: flexgrid_checkbox_process,
    width : 25,
    sortable : false,
    align: 'left'
  },
  {
    display: 'Name',
    name : 'name',
    width : 150,
    sortable : true,
    align: 'left'
  },
  ...
]

The process function receives two parameters (flexgrid.js, line 700:)
----
    if (pth.process) pth.process(tdDiv,pid);
----
tdDiv at that moment is not attached to document tree so there's no easy way to 
find the owner of this div.

What I'd suggest is add third parameter to that function call that points to 
flexgrid containerd div, ie:
----
    if (pth.process) pth.process(tdDiv,pid,g.gDiv);
----
Benefits? I can have multiple flexgrids in the same page + do things like 
saving state of checkboxes across different pages since now I know which 
flexgrid owns the cell being processed.

Original issue reported on code.google.com by tadas.sa...@gmail.com on 24 Jan 2011 at 12:40