replace onload with addEventListener in inject_styles

[ankitstarski]

When multiple dynamically imported components are loaded, only one of them resolves the promise. Upon further investigation, I found out that those components were loading .css files and inject_styles function was using onload in order to resolve inject_styles promise.

If two components use the same css file, the first onload will be overridden by the second one.

The solution is to replace onload with addEventListener("load", ....

@Rich-Harris pushed a fix for this in svelte kit (kit#347)

[benmccann]

I'm not merging much into Sapper, but this one looks fine since it aligns the code with SvelteKit

[benmccann]

Although that code change was from before SvelteKit switched to Vite. That code no longer exists in SvelteKit as Vite is now responsible for it. That being said I'll go ahead and merge this since it looks like Rich approves of the change