Page 2 of 2

Re: Disabling onrowdblclick event

Posted: Tue May 14, 2013 12:43 pm
by emhill
Thanks, Scott. This did just seem to happen after we went to 4.5.2. Didn't have any complaints before then.

Re: Disabling onrowdblclick event

Posted: Thu May 16, 2013 11:12 am
by Scott Klement
I compared our code from last November to the current code, and nothing has changed with regard to this.

When I did some troubleshooting, it seems that newer versions of IE and Firefox were telling PUI that you had clicked on an HTML "OPTION" tag. Older versions of browsers said you were clicking on an HTML "SELECT" tag.

As you might know, a drop-down looks like this (in HTML):

Code: Select all

<select (parameters here)>
   <option>First Value</option>
   <option>Second Value</option>\
</select>
So technically, you were clicking both SELECT and OPTION. But all of the older browsers report "SELECT", and the newer Firefox and IE9/10 report "OPTION". So it would seem that the browsers have changed.

I simply changed the code to look for either SELECT or OPTION, and that seemed to solve the problem. I've sent you a patch, let me know if it works for you, too.

Re: Disabling onrowdblclick event

Posted: Fri May 24, 2013 3:18 pm
by Karthik
Hi Scott,

I get an Onrowdblclick Error: when I double click on an empty subfile record. Message says "Unable to get value of the property 'disabled': object is null or undefined".

Re: Disabling onrowdblclick event

Posted: Tue May 28, 2013 1:26 pm
by Scott Klement
You are doing the following, correct?

Code: Select all

var dropdown = getObj("Dropdown1." + row);
if (dropdown.disabled) {
   // do your double click logic here.
}
If so, it would seem that some of your rows don't have a "dropdown1" widget in them. Am I right?

If so, perhaps you need to do something like this?

Code: Select all

var dropdown = getObj("Dropdown1." + row);
if (dropdown!=null && dropdown.disabled) {
   // do your double click logic here.
}

Re: Disabling onrowdblclick event

Posted: Fri Jun 07, 2013 8:15 am
by Karthik
Exactly what I needed. Thanks Scott.