Skip to content

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

New issue

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

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Comments

@jefferyto
Copy link
Contributor

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.)

@scottgonzalez
Copy link
Member

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.

@jefferyto
Copy link
Contributor Author

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?

@scottgonzalez
Copy link
Member

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.

@scottgonzalez
Copy link
Member

Closing due to inactivity.

arthurvr pushed a commit to arthurvr/learn.jquery.com that referenced this issue Jan 4, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

2 participants