Always visible editing panel - select multiple

Tronik

Im using the Always visible editing panel plugin, which works great
However, it doesn't seem support multi-row edits.

How can I make that work?

tangerine

I just made a multi-edit on that example.
Why do you say it doesn't work?

Tronik

Sorry, my mistake, it works now. Dont know why it didn't at first.

colin

Yep, I see what you mean - you're both right. If you shift-click to select a range it works, but, if you ctl-click to select several, it doesn't. I'm not sure why that's the case, we'll take a look,

Colin

Tronik

Yes, that seems to be the case.

Tronik

I cant find an "un-answered" button

allan

I've removed the answered flag now.

The issue appears to be caused by a bug in the Select extension when using ctrl-click to select rows. You can see it in this example by running the following code in the console:

$('#example').DataTable().on( 'select', function ( e, dt, type, indexes ) {
  console.log('indexes:', indexes);
});

We'll get this filed in a bug report and report back here when it is fixed.

Allan

allan

Just thought of a workaround until we get this fixed. Instead of passing in indexes to the edit() method call, pass in dt.rows({selected:true}).indexes().

Allan

Tronik

I’ll try that!

Many thanks

allan

Little update on this. I implemented the change required, but it broke our unit tests, and caused us to re-evaluate this change. We think that the indexes parameter passed it should reflect just the newly selected rows and anything else would break backwards compatibility.

So the workaround we discussed above is actually the correct solution. I'm going to update the blog post to reflect that.

Allan

Tronik

Ok, thanks!