-
Notifications
You must be signed in to change notification settings - Fork 480
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
Comments
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. |
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 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? |
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. |
…is addresses the first half of jquery#409)
Closing due to inactivity. |
…is addresses the first half of jquery#409)
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.)
The text was updated successfully, but these errors were encountered: