Skip to content

Make knowls wait until the page css has loaded before initializing.#696

Merged
pstaabp merged 1 commit intoopenwebwork:PG-2.17from
drgrice1:knowls-await-load
Jul 13, 2022
Merged

Make knowls wait until the page css has loaded before initializing.#696
pstaabp merged 1 commit intoopenwebwork:PG-2.17from
drgrice1:knowls-await-load

Conversation

@drgrice1
Copy link
Member

This fixes issue webwork2#1739.

In the PG problem editor, the page in the iframe is not displayed until
after the page loads. As such the javascript getComputedStyle method
fails. The same things happens with the graphtool. The same solution
is now implemented for knowls.

This fixes issue openwebwork/webwork2#1739.

In the PG problem editor, the page in the iframe is not displayed until
after the page loads.  As such the javascript getComputedStyle method
fails.  The same things happens with the graphtool.  The same solution
is now implemented for knowls.
@drgrice1
Copy link
Member Author

When testing this make sure to remember to run npm install or ./generate-assets in the pg/htdocs directory.

Copy link
Contributor

@somiaj somiaj left a comment

Choose a reason for hiding this comment

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

I can confirm this makes solution knowls work for me again in the problem editor.

@mgage mgage requested a review from somiaj July 13, 2022 17:07
@mgage
Copy link
Member

mgage commented Jul 13, 2022

This patch works for me also

@pstaabp pstaabp merged commit 3ef38fe into openwebwork:PG-2.17 Jul 13, 2022
@drgrice1 drgrice1 deleted the knowls-await-load branch July 13, 2022 19:27
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