jQuery Grid Plugin - jqGrid - Topic: onSelectRow not called on checkboxes (formatter:'checkbox')

OlegK on onSelectRow not called on checkboxes (formatter:'checkbox')

Sun, 14 Aug 2011 13:00:48 +0300

Hi, Tony!

It seems that there are differences how different web browsers works with disabled elements. If you look at the demo in IE you you will be able to make double-click on the disabled checkbox. So the "click" event which I describe will be fired in IE in case of double-click. The seme demo in Chrome not works because Chrome don't call "click" event on disabled element.

So my suggestion can improve the situation a little, but not really solve it.

Best regards
Oleg

P.S. I am glad to see you back.

tony on onSelectRow not called on checkboxes (formatter:'checkbox')

Sun, 14 Aug 2011 12:53:06 +0300

Oleg,

Not sure I understand the demos here.

Allow click on disabled input element is maybe good idea.

Not sure what to do with the returns. Will make some experiments.

Tony

OlegK on onSelectRow not called on checkboxes (formatter:'checkbox')

Thu, 30 Jun 2011 21:26:18 +0300

Hello Tony,

if one try to implement inline editing on ondblClickRow one could recive strange problem: the onSelectRow event will not fired if the user click on the disabled checkbox created by formatter:'checkbox'. See the demo. As a workaround one have to use beforeSelectRow instead of onSelectRow (see the fixed demo).

The problem make the line 2211 of the grid.base.js:

if (td.tagName == 'A' || ((td.tagName == 'INPUT' || td.tagName == 'TEXTAREA' || td.tagName == 'OPTION' || td.tagName == 'SELECT' ) && !scb) ) { return this; }

It seems to me in the line one should include additional test for td.disabled (or use :not(:disabled) selector). So one should do allow to select lines by clicking on the disabled inpit elements.

One more remark. Is it correct to use return this inside of click event handler (it is here and in the another line above)? One should not stop the event.

Best regards
Oleg