WebGazer script needs to be added on body not head about webgazer

[jeffhuang]

Any progress on this issue? This seems to be a tough one to track one.

[Jack-Wong94]

It appears the webgl is not properly set up when the webgazer.js is in the head. It is because the browser will download the javascript and proceed down the line of the html. If the javascript url reference in the head gets stuck it would cause the issue. I have add an async tag when putting the script reference into and it seems to solve the problem.

[jeffhuang]

Ahh good find! That makes sense. Would async still occasionally cause problems since it's sort of introducing a race condition? Do you have a suggestion to make sure that developers do this properly to avoid this bug in the future?

[Jack-Wong94]

Using async will excute the script while the page continues parsing. From what I see, it doesn't cause problem regarding switching to the slower version of javascript. But if the race condition does cause problem, they can use defer instead of async which would parsing the page first before excuting webgazer.js. It would still solve the problem but also eliminate the potential of introducing a race condition.

[jeffhuang]

Got it, async sounds like a good solution for now. defer might make things load a bit slower, but might be good to mention as well. Can we add a note about using async/defer in the documentation/readme?

[Jack-Wong94]

Yes, we could add the note into the readme

[idaknow]

Added a note to the README regarding this