Skip to content

Begin work on converting XSLT data#2026

Merged
Elchi3 merged 3 commits intomdn:masterfrom
ExE-Boss:xslt/elements/stylesheet
Aug 27, 2018
Merged

Begin work on converting XSLT data#2026
Elchi3 merged 3 commits intomdn:masterfrom
ExE-Boss:xslt/elements/stylesheet

Conversation

@ExE-Boss
Copy link
Contributor

@ExE-Boss ExE-Boss commented May 15, 2018

This PR begins work on converting XSLT data, which consists of a total of 35 elements.

needinfo?(@jpmedley): Does Chrome and derivates support this?


Version information sources:


See also mdn/kumascript#734

review?(@Elchi3)

@ExE-Boss ExE-Boss mentioned this pull request May 15, 2018
38 tasks
Copy link
Contributor

@maboa maboa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The URLs seem to be incorrect - also unsure where you are getting the data from. I'm not seeing any tables.

"elements": {
"stylesheet": {
"__compat": {
"mdn_url": "https://developer.mozilla.org/docs/Web/XSLT/Element/stylesheet",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This link is pointing to a 404 - suspect the /Element/ is not required.

Copy link
Contributor Author

@ExE-Boss ExE-Boss May 22, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See #2027, I’d move it myself, but I don’t yet have the createpages MDN permission, which would allow me to move the pages to the correct locations.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Update: I’ve now moved the page to the new location.

@ExE-Boss
Copy link
Contributor Author

ExE-Boss commented May 22, 2018

unsure where you are getting the data from. I'm not seeing any tables.

Gecko (Firefox) is documented on the page (it’s just not in a table, so it requires actual reading), and I’ve tested iOS, IE and Edge myself (I refuse to install Chrome, so I can’t test it, which is why I asked for @⁠jpmedley).

"mdn_url": "https://developer.mozilla.org/docs/Web/XSLT/Element/stylesheet",
"support": {
"chrome": {
"version_added": null
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ExE-Boss ExE-Boss force-pushed the xslt/elements/stylesheet branch 2 times, most recently from 9eb33a0 to 092cb69 Compare June 12, 2018 02:06
@ExE-Boss ExE-Boss mentioned this pull request Aug 20, 2018
2 tasks
Copy link
Member

@Elchi3 Elchi3 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me. Let's do this! 👍 🎉

@Elchi3 Elchi3 merged commit 3273f60 into mdn:master Aug 27, 2018
@ExE-Boss ExE-Boss deleted the xslt/elements/stylesheet branch August 27, 2018 13:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants