search

samturrell / vue-breadcrumbs

Breadcrumbs for Vue.js
MIT License
146 stars 27 forks source link

Vue-breadcrumbs breaks webpage on IE 10 #13

Closed marcosPerezRubio closed 7 years ago

marcosPerezRubio commented 7 years ago

Hello!

As vue-breadcrumbs uses Object.assign, this causes IE 10 to not load the webpage because it can not support it.

This is easily solvable by adding the babel polyfill (https://babeljs.io/docs/plugins/transform-object-assign/).

Could this be implemented in a future release?

Thanks!

samturrell commented 7 years ago

Hi @marcosPerezRubio - I would probably say it's better to add the polyfill to your implementation rather than it being added to the core of the plugin as it would create unnecessary overhead for browsers that have native support.

marcosPerezRubio commented 7 years ago

Agree with that!

Thanks for your response :)

Marcos Pérez

El 19 nov. 2017 14:06, "Sam Turrell" notifications@github.com escribió:

Hi @marcosPerezRubio https://github.com/marcosperezrubio - I would probably say it's better to add the polyfill to your implementation rather than it being added to the core of the plugin as it would create unnecessary overhead for browsers that have native support.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/samturrell/vue-breadcrumbs/issues/13#issuecomment-345515509, or mute the thread https://github.com/notifications/unsubscribe-auth/AKT74cLLiFRxvnNfKL2xV38ASSWBXYwHks5s4Ce_gaJpZM4QEMYY .