Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Advanced Plugin Concepts: Replace "Support the Metadata Plugin" with discussion on HTML5 data- attributes #409

Closed
jefferyto opened this Issue Aug 7, 2013 · 4 comments

Comments

Projects
None yet
2 participants
Contributor

jefferyto commented Aug 7, 2013

Since the Metadata plugin has been officially deprecated for over 2 years (and now orphaned by the jQuery Foundation), it would be good to replace the "Support the Metadata Plugin" section of Advanced Plugin Concepts with a discussion on a plugin can support HTML5 data- attributes.

(EleventyOne at StackOverflow noticed this issue, I just decided to file a ticket here.)

Owner

scottgonzalez commented Aug 7, 2013

I'd prefer if that entire section is just removed. There should be a section explaining how to implement a simplified jQuery Mobile attribute parser using the widget factory.

Contributor

jefferyto commented Aug 12, 2013

I haven't read through all of the content, but it seems like that page is geared towards folks who want to build a plugin "from scratch". If so, using data- attributes seems like a good fit.

Perhaps a section on building an attribute parser using the widget factory would be better suited in the Widget Factory section?

I think it would be good to at least remove the outdated Metadata plugin section first, then decide what to replace it with? Should I submit a pull request for that?

Owner

scottgonzalez commented Aug 12, 2013

Yeah, I think starting with a PR to remove it is a good first step. The reason I don't think it's a good idea to add anything back is because if you're reading data attributes, you're probably building a stateful widget. And if you're doing that, you should be using the widget factory, not building from scratch.

@jefferyto jefferyto added a commit to jefferyto/learn.jquery.com that referenced this issue Aug 26, 2013

@jefferyto jefferyto Removed the Metadata plugin section from Advanced Plugin Concepts (th…
…is addresses the first half of #409)
1abf45e
Owner

scottgonzalez commented Apr 10, 2014

Closing due to inactivity.

@arthurvr arthurvr pushed a commit to arthurvr/learn.jquery.com that referenced this issue Jan 4, 2015

@jefferyto jefferyto Removed the Metadata plugin section from Advanced Plugin Concepts (th…
…is addresses the first half of #409)
ae2ebc0
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment