FR: YouTube / Google Maps should load the API itself

[vandres]

Feature Description

I wish the new Youtube package would load the API itself, so we can use the component directly without wrapping it in our own component.

[IgorMinar]

I also find it unexpected that the player requires app developers to manually load the youtube api. Maybe provide a .forRoot() or .init() static method on the YouTubePlayerModule that will optionally download the youtube player api.

[guillaume-ro-fr]

I also find it unexpected that the player requires app developers to manually load the youtube api. Maybe provide a .forRoot() or .init() static method on the YouTubePlayerModule that will optionally download the youtube player api.

Maybe a .forChild() method, imported in modules will be better for lazy-loaded routes/modules ? This method could check the YouTube loading with the YT global variable I guess.

[gioragutt]

@IgorMinar is your preferred approach to add .forRoot() method to the YouTubePlayerModule?
If so, I'd love to make a PR.

[jelbourn]

reposting my comment from Kristiyan's PR

I'm still quite leery of adding an API that loads the API for you. My rationale is along the lines of:

• This isn't something we do for any other dependency.
• I generally think developers should be aware of how they're loading deps into their application, and I'm not a fan of having a black-box API that obscures that.
• Having an API for loading the dependency implies that there's a "correct" way to do it, when it depends somewhat on what the app wants to do.
• It's easier for apps to end up in a state with multiple versions of the dependency loaded.
• Such an API would inherently modify browser global state, which could be problematic in instances where applications want to bootstrap multiple Angular applications. I know we haven't been the best about this before, so it's something that comes to mind now.

The thing I don't have a strong insight into is how painful this is for application developers. To me it seems normal/expected/straightforward to manage a dependency like this, but it's possible I'm missing some nuance here.