Uh oh!
There was an error while loading. Please reload this page.
ref(vue): Use string.repeat() - #6663
Conversation
There was a problem hiding this comment.
Lol, never saw this before xD. I think this should be fine. If users bundle their app and they want to support IE11, they could just polyfill String.repeat
Also, this is only used for creating a component "trace" (overloaded term...) in event.context.vue of error events, so the impact shouldn't be too drastic
size-limit report 📦
|
mydea
commented
Jan 5, 2023
Yeah, I thought so as well. And we can save a few precious bytes ;) |
I stumbled over this by accident, and was wondering what you thought about making use of the in-built
string.repeat()method there. This is not supported in IE11, but IMHO it is acceptable to fall back to just use the string once in that case, as it will only result in less-nice-looking nesting, not break. But not sure, maybe it is a bad idea, WDYT?