fedwiki / wiki-plugin-video

Embed Video from YouTube or Vimeo
Other
3 stars 5 forks source link

Paul90/page update #5

Closed paul90 closed 9 years ago

WardCunningham commented 9 years ago

We still have some trouble redundantly loading plugins even though we check to see if we have loaded them. This comes from rendering multiple plugins at once. Each one discovers that it doesn't have the needed plugin and then issues its own request.

I think this load logic suffers from the same problem. On a page with three videos, if it decides to load the css it will probably load it three times. I'm merging this code anyway because proper spacing is more important than saving every possible server hit.

paul90 commented 9 years ago

We still have some trouble redundantly loading plugins even though we check to see if we have loaded them.

Do we still have that problem? I don't see it with my local test server, so probably caused by latency if we do.

If it is still a problem, we probably need to write some code that can act as a central agent to load the dynamic elements. This would need to be aware of not only what is already loaded, the simple bit, but also what has been requested and is not yet loaded.