jQuery Forum

1. If you're adding a new option called scrolling you need to extend $.ui.dialog.prototype.options and give it a default value.

2. It looks like you're trying to create this scrolling option in the browse method:

1.     $.ui.dialog.prototype.browse = function (url, scrolling) {
2.         console.log('browse', arguments);
3.         if (url) this.options.url = url;
4.         if (scroll) this.options.scrolling = scrolling === true ? 'yes' : scrolling === false ? 'no' : scrolling || 'auto';

but the 'scroll' in if (scroll) is undefined. Did you mean if (scrolling)?

3. You have no _setOption handling. While this isn't needed for a scrolling option (if one exists) to get a new value, it is required to handle any necessary side effects for when the value changes.

Thanks for the quick reply