Skip to content
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

Menu Bug? #2216

Closed
mewcrazy opened this issue Feb 10, 2020 · 6 comments
Closed

Menu Bug? #2216

mewcrazy opened this issue Feb 10, 2020 · 6 comments

Comments

@mewcrazy
Copy link

mewcrazy commented Feb 10, 2020

Hello,

I just found a bug with the menu list? My markup looks as follows: I already tried removing the showJobs function to see if that's the reason for the thrown error, but it's not.

<b-menu> <b-menu-list> <b-menu-item class="mb-2 bg-purple" icon="user" label="Alle" @click="showJobs(0)"></b-menu-item> <b-menu-item class="mb-2 bg-transwhite80" icon="user" label="Freelancer.de"></b-menu-item> <b-menu-item class="mb-2 bg-transwhite80" icon="user" label="Interlance.de" @click="showJobs(2)"></b-menu-item> </b-menu-list> </b-menu>
Vue: ^2.6.11
Buefy: ^0.8.10

@mewcrazy
Copy link
Author

image

@jtommy
Copy link
Member

jtommy commented Feb 11, 2020

Same issue #2094

@service-paradis
Copy link
Collaborator

service-paradis commented Feb 11, 2020

Related to this Vue problem (since 2.6.11):
vuejs/vue#10939

@mewcrazy
Copy link
Author

mewcrazy commented Feb 11, 2020

Sounds relatively easy to solve though: Just remove the .native? :D I'm not even sure if the .native still work in Vue2, I tried to use them somewhere else with keypress enter and .native and they did not have any effect. As soon as I removed the .native my code worked flawlessly without any sideeffects.

@service-paradis
Copy link
Collaborator

service-paradis commented Feb 11, 2020

Not that easy since the .native is needed when the tag used is not a a.
The easiest workaround right now is to downgrade Vue to 2.6.10.

jtommy added a commit that referenced this issue Feb 16, 2020
@jtommy
Copy link
Member

jtommy commented Feb 16, 2020

Fix 4a65d76

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants