-
-
Notifications
You must be signed in to change notification settings - Fork 8.4k
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
feat: expand defaultAssetUrlOptions
#9894
base: minor
Are you sure you want to change the base?
Conversation
Expand defaultAssetUrlOptions to include other asset-linking html attributes
It seems odd to me that the I don't see a reason why the attributes added in this PR were omitted in the first place. If there's a reason or a potential drawback to this change, I'd like to know! |
Size ReportBundles
Usages
|
CodSpeed Performance ReportMerging #9894 will not alter performanceComparing Summary
|
defaultAssetUrlOptions
I believe this would make #7914 redundant. A couple of places that might also need updating if this is merged:
Should this change target |
@vue/compiler-core
@vue/compiler-dom
@vue/compiler-sfc
@vue/compiler-ssr
@vue/reactivity
@vue/runtime-dom
@vue/runtime-core
@vue/shared
@vue/server-renderer
@vue/compat
vue
commit: |
Expanded defaultAssetUrlOptions to include other asset-linking attributes