Bug Tracker

Modify

Ticket #10442 (closed feature: wontfix)

Opened 20 months ago

Last modified 15 months ago

Integrate widget factory

Reported by: jzaefferer Owned by:
Priority: undecided Milestone: 1.next
Component: core Version: git
Keywords: 1-8-discuss Cc:
Blocking: Blocked by:

Description (last modified by jzaefferer) (diff)

This ticket is mostly meant to start a discussion around the jQuery UI widget factory, and potentially integrating it into jQuery Core to make it more accessible to (plugin) developers. Even in the likely case of not landing it, getting feedback on the current design from core developers would be very useful.

Current version is here, there is a bunch of new features going to land in 1.9:  https://github.com/jquery/jquery-ui/blob/master/ui/jquery.ui.widget.js

Unit tests are here:  https://github.com/jquery/jquery-ui/tree/master/tests/unit/widget

Documentation (wip) is here:  https://github.com/scottgonzalez/widget-factory-docs (doesn't yet cover the internal API, like _create, _bind and more).

Change History

comment:1 Changed 20 months ago by jzaefferer

  • Keywords 1-8-discuss added
  • Description modified (diff)
  • Milestone changed from None to 1.8

comment:2 Changed 20 months ago by scott.gonzalez

I'm not really sure what we'll get out of this. John did a review a while ago and I incorporated all of his feedback in the 1.8 release.

comment:3 Changed 20 months ago by dmethvin

  • Status changed from new to open
  • Component changed from unfiled to core

Moving to open for 1.8 discussion.

comment:4 Changed 16 months ago by dmethvin

  • Milestone changed from 1.8 to 1.next

We're not planning to do this in 1.8.

comment:5 Changed 16 months ago by davidsouther@…

What about the discussion of moving $.widget into core? Alone, $.widget is a very powerful tool for creating plugins, and has almost no dependencies on anything else in UI. It could be very useful to pull the hierarchy and bridge code out, say into core jquery.plugin.js, and leave jquery.widget.js with the $.Widget code, as that is specific to UI plugin lifecycles.

comment:7 Changed 16 months ago by dmethvin

We don't require either in core, and our emphasis with 1.8 is to reduce bloat. Users can easily combine whatever plugins they require into a single file for their own project needs.

comment:8 Changed 15 months ago by dmethvin

  • Status changed from open to closed
  • Resolution set to wontfix

I don't see any good reasons for putting this in core, since we don't have a dependency on it, so I'll close this ticket.

Please follow the  bug reporting guidlines and use  jsFiddle when providing test cases and demonstrations instead of pasting the code in the ticket.

View

Add a comment

Modify Ticket

Action
as closed
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.