-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
Fix: support for using require.js in pure XML #1559
base: master
Are you sure you want to change the base?
Conversation
I am a bit hesitant to do too many changes to support pure XML documents, I just do not want to guarantee full support since I am not confident there are enough tests for that environment. What about the following instead, to avoid too many changes in the file:
How does that sound? |
I definitely agree with |
The proposed fix of using !document.body for this XML case results in using The other solution if there were very specific needs of an XML document is for it override Using requirejs in XML documents has not been a popular request, given how long the project has existed and the code has existed in its current form for so long. So I do not want a fix to get too involved for such a long tail use. |
So just to be clear you do not want to fix this issue at all, although the Le lun. 11 juil. 2016 23:34, James Burke notifications@github.com a
|
If the pull request is updated to match the 1) and 2) items above, then I am open to merging. I'll be traveling with bad internet for the next few days, so I my next response on this issue will be delayed. |
|
require.js
does not work with pure-XML documents right now. This patch fixes it.vs