The array temp_new_rows is used to keep track to new added rows in View data. It is used to add 'disabled_row'class for new rows to highlight when grid renders after delete/copy/add row operation.
On Mon, May 15, 2017 at 9:28 PM, Dave Page <dpage@pgadmin.org> wrote:
Aww, nuts. I thought we had tests for that? Did we miss that particular case?
Anyway, copying of a single value still works (with Cmd+C), so I'm not inclined to re-wrap everything for this unless someone can get a patch to me by ~9AM UK time tomorrow. Surinder?
We were checking out the selection functionality after this commit--looks like selection no longer works for single cells.
If you press a cell in the grid the following message is console logged:
sqleditor.js:869 Uncaught TypeError: Cannot read property 'indexOf' of undefined at SlickGrid.<anonymous> (sqleditor.js:869) at Event.notify (slick.core.js:143) at trigger (slick.grid.js:1067) at setActiveCellInternal (slick.grid.js:2693) at HTMLDivElement.handleClick (slick.grid.js:2469) at HTMLDivElement.dispatch (jquery-1.11.2.js:4665) at HTMLDivElement.$event.dispatch (jquery.event.drag-2.2.js:374) at HTMLDivElement.elemData.handle (jquery-1.11.2.js:4333)
Thanks
George & Joao
On Mon, May 15, 2017 at 10:05 AM, Dave Page <dpage@pgadmin.org> wrote:
I've committed a modified version of this patch following some discussion on IM with Surinder.