jQuery Grid Plugin - jqGridForum

Topic RSS

Related Topics sp_TopicIcon

exception in click event handler of the cell click

Tags: onCellSelect onselectRow

20/01/2012
00:53

OlegK

Germany

Member

Members

Forum Posts: 1255

Member Since:
10/08/2009

Offline

Hello Tony,

I sent one more pool request. It fixs the bug in the event handler of the click on the grid cell. The problem is that if the call has more complex structure and the onSelectRow callback calls editRow the exception can be fired.

One can reproduce the bug in IE on the demo. Exception will be in the line. If one clicks on the checkbox one will see that during calculation of $.inArray(c, c.parentNode.cells)

c = (!c.is('td') && !c.is('th') ? c.closest("td,th") : c)[0];
if ($.browser.msie) { return $.inArray(c, c.parentNode.cells); }

the c is undefined. The problem is that the line will be called after setSelection which calls onSelectRow, which calls editRow in my demo. The editRow method change the content of the clicked cell. After that the e.target.parentNode of the original click event and even window.event.srcElement.parentNode will be set to null.

As a workaround I suggest to change the order of executing onCellSelect and jqGridCellSelect and calling of onSelectRow callback or editCell.

The same demo which use the fixed version you can test here.

Best regards
Oleg

20/01/2012
18:27

tony

Sofia, Bulgaria

Moderator

Members

Moderators

Forum Posts: 7721

Member Since:
30/10/2007

Offline

Thanks Oleg,

I do not see it as Pull in GitHUb.

Instead I have apply the fix.

Regards

Tony

For professional UI suites for Java Script and PHP visit us at our commercial products site - guriddo.net - by the very same guys that created jqGrid.

20/01/2012
18:38

OlegK

Germany

Member

Members

Forum Posts: 1255

Member Since:
10/08/2009

Offline

Hello Tony,

I had problems to create new pool request, because the old one already exists. So all new requests are placed (appended) in the same old pool request (see here). I don't know whether you can apply only one part from the request.

I posted last time many suggestions to jqGrid which could be interesting for you (in my opinion). Just now I posted the answer which I recommend you to read. The fixes which I suggested in the answer on the stackoverflow allows with some restriction use inline editing. I think it could be interesting for many jqGrid users.

Best regards
Oleg

P.S. Your comments to my last suggestion are very intresting for me. What you think about this, this and this bug reports and about this, this, this and this feature requests?

All RSS

Forum Timezone: Europe/Sofia

Most Users Ever Online: 715

Currently Online:
33 Guest(s)

Currently Browsing this Page:
1 Guest(s)