Just one note.

I think that this part is incorrect: “The astute reader will have noticed that level is a class variable; the same variable is used for every green3 object. This is clearly not what we want; each instance should have its own copy”

It looks to me that jQuery UI create an instance of our prototype for each object. So you coul safely use this.level, and it will be a different variable for each object.

I noticed this because some jQuery UI widgets just store variables using this.name_variable.
For instance, the accordion widget save the headers elements as “this.headers”

Thanks anyway for this great article, it’s so usefull that there should be an official copy on the jQuery UI doc

_Valentino

br
rob

I added an explicit trigger change to the hide function – seems to be working

great widget!!!
Brian

for what it’s worth, it’s line 122 in v1.7.2 of ui.dialog.js
from

.text(options.closeText)
to
.text((options.closeText || ”)),

maybe you can explain why exactly extension fails on this internally???

i understand that .text(undefined) is returning a null therefore .appendTo() doesn’t exist and error, but what’s going on here under the hood?

Thanks for the great article, i’ve learned tons tonight. With this small fix i’m able to extend the ui.dialog also

timepickr seems to be eating the change event. $(aField).change works fine when I also attach timepickr $(aField).timepickt(…) the change event does not trigger

Brian

yes, exactly thanks

good spotting