-
Notifications
You must be signed in to change notification settings - Fork 73
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
[fix] tab link break routes #29
[fix] tab link break routes #29
Conversation
Hi @ahoendgen Thanks for pointing this out. There is one issue with the PR however. The modification includes only the built |
dist/vue-tabs.common.js
Outdated
@@ -195,6 +195,12 @@ var VueTabs = { | |||
[_this.textPosition === 'top' && _this.renderTabTitle(index, _this.textPosition), h( | |||
'a', | |||
{ | |||
on: { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is a built file. Please do not commit it but rather modify the files inside src
folder and commit them instead.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yep, my fault, i will change this.
btw. is there a reason you didnt add the babel-vue-app-preset to dev-dependencies? |
f488702
to
7b17beb
Compare
@ahoendgen you could add that preset to dev deps. |
@cristijora already did that on my machine, added this also to this mr now |
Great! Thanks. Will release this today. |
If you use hash based routes the link on each tab will break the routing, this can be easily fixed with preventing the event to be executed.