Opened 8 years ago

Closed 8 years ago

#8141 closed enhancement (wontfix)

Cache raw un-merged options from instantiation in widget for more complicated post-processing

Reported by: fooblah Owned by: fooblah
Priority: minor Milestone: 1.9.0
Component: ui.widget Version: git (not yet released)
Keywords: Cc:
Blocked by: Blocking:

Description

I added a line:

this.rawOptions = options;

in _createWidget so that I could handle merging some options a special way.

Specifically, I wanted certain arrays nested in user provided options to replace instead of merge.

Change History (3)

comment:1 Changed 8 years ago by Scott González

Owner: set to fooblah
Status: newpending

In master (1.9), we never merge arrays. We don't even clone, we reference the original array. How does this affect your use cases?

comment:2 Changed 8 years ago by fooblah

Status: pendingnew

Sounds good. I missed that because at https://github.com/jquery/jquery-ui/blob/master/ui/jquery.ui.widget.js#L210 I missed you guys were calling $.widget.extend and not $.extend.

Thanks.

BTW, is master solid to use for production work yet? I am only using core and widget.

comment:3 Changed 8 years ago by Scott González

Resolution: wontfix
Status: newclosed

Ok, if that handles your use cases, then I'll close this as wontfix.

Master is pretty stable. It should actually be more stable than our stable releases.

Note: See TracTickets for help on using tickets.