-
Notifications
You must be signed in to change notification settings - Fork 12k
Deprecate styles and script lazy
option in favor of a more self descriptive option
#14814
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
@filipesilva any thoughts about the above? We discussed this in our meeting and myself, @mgechev and @clydin all think it's best to deprecate We didn't yet decide the name of the new option though, May it can be |
Deprecation+migration on 9 SGTM. I have no opinion on Looking at |
Discussed and decided on |
…zy` option in favor of`inject` The lazy option inside the script and style option is confusing as this option doesn't lazy load a bundle but rather it doesn't inject/reference the script in the HTML. While this option is an enabler for lazy loading, the users will still need to handle on how how this bundle will be lazy loaded. There are also potential use cases beyond lazy loading for the option. Closes #14814
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
🚀 Feature request
Description
The
lazy
option inside thescript
andstyle
option is confusing as this option doesn't lazy load a bundle but rather it doesn't inject/reference the script in the HTML. While this option is an enabler for lazy loading, the users will still need to handle on how how this bundle will be lazy loaded. There are also potential use cases beyond lazy loading for the option.Deprecate the
lazy
option in favour of a more self descriptive options such asinjected
.The text was updated successfully, but these errors were encountered: