Skip to content
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

Adjust ADD_JS_FILE handling code to deal with new third parameter #1374

Merged
merged 1 commit into from
May 27, 2021

Conversation

drgrice1
Copy link
Member

Adjust the code for dealing with the files added by PG.pl's ADD_JS_FILE method to add attributes provided in the new optional third argument.

The code is also adjusted to ensure that scripts are loaded in the order that they are requested, instead of in the random order that perl gives with hash keys. Should this be done with the ADD_CSS_FILE handling code also? Order matters for css loading as well.

This is paired with openwebwork/pg#576.

method to add attributes provided in the new optional third argument.

The code is also adjusted to ensure that scripts are loaded in the order
that they are requested, instead of in the random order that perl gives
with hash keys.  Should this be done with the ADD_CSS_FILE handling code
also?  Order matters for css loading as well.
Copy link
Member

@taniwallach taniwallach left a comment

Choose a reason for hiding this comment

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

Code looks fine, but it would be nice if someone tested in all 3 contexts: homework, gateway, and html2xml, and verified that order is maintained, and that the 3-rd option is working for all 3 contexts.

@pstaabp pstaabp merged commit 8cafe90 into openwebwork:WeBWorK-2.16 May 27, 2021
drgrice1 pushed a commit that referenced this pull request May 27, 2021
Adjust ADD_JS_FILE handling code to deal with new third parameter
@drgrice1 drgrice1 deleted the add_js_attribs branch May 27, 2021 17:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
NeedsTesting Tentatively fixed bug or implemented feature priority2 (moderate)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants