Tony Gray

Results 35 comments of Tony Gray

see: www.slickgrid.net Main GitHub Repository is now: github.com/6pac/SlickGrid

This is not the most elegant fix but replace the rowselectionmodel line 123 if (active >= 0 && active < _grid.getDataLength()) { _grid.scrollRowIntoView(active); var tempRanges = rowsToRanges(getRowsRange(top, bottom)); setSelectedRanges(tempRanges); }...

New version ![2019-02-25_11-28-33](https://user-images.githubusercontent.com/2100563/53306333-b3b80980-38f0-11e9-9bcd-17fd4436827a.gif) Old version did ![2019-02-25_11-29-21](https://user-images.githubusercontent.com/2100563/53306342-da764000-38f0-11e9-954e-97794b570422.gif) This is without holding CTR or Shift which still do the correct funcitonality

@ghiscoding Yep I think it's before the update for the frozen features. Also it doesn't seem to work how expected which was what i've shown in the gifs. Original you...

@ghiscoding Here is my example of how it worked / works with my changes. MultiSelect = true, `selectActiveRow` = true ![2019-02-25_16-29-07](https://user-images.githubusercontent.com/2100563/53312628-af084b00-391a-11e9-8861-563c6625fd61.gif) Here you can see select checkboxes allows multi select...

@ghiscoding @6pac It has changed how the code used to work here. By all means go and change to the old version and check if you want. The functionality of...

@ghiscoding Just checked you seem to have removed the focus which fixes the `Crtl` key double click part of issue. However may pay to do the fix to the `slick.rowselectionmodel.js`...

@ghiscoding Did you end up adding the stuff to the slick.rowselectionmodel.js? Then we can close this one

@ghiscoding Interesting that yours isn't working from checkboxes for {multiSelect: true, selectActiveRow: true} With the pull #346 change id expect Ctrl + click on row and click on checkboxes to...

@ghiscoding @6pac The fix here just fixes up the single select version. The multi works how I want for me at the moment so don't want to change it. If...