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

[Bug Report] v-tooltip and v-menu do not work with an external activator #5890

Closed
builder7777 opened this issue Dec 13, 2018 · 6 comments · Fixed by #8121
Closed

[Bug Report] v-tooltip and v-menu do not work with an external activator #5890

builder7777 opened this issue Dec 13, 2018 · 6 comments · Fixed by #8121
Labels
C: VMenu VMenu C: VTooltip VTooltip S: has PR The issue has a pending Pull Request T: bug Functionality that does not work as intended/expected
Milestone

Comments

@builder7777
Copy link

Versions and Environment

Vuetify: 1.3.14
Vue: 2.5.21
Browsers: Chrome 70.0.3538.110
OS: Mac OS 10.13.6

Steps to reproduce

Create v-tooltip component, create a separate activator button.

Expected Behavior

Tooltip should appear when hovering over activator

Actual Behavior

Tooltip does not appear

Reproduction Link

https://codepen.io/anon/pen/roxwoZ?&editable=true&editors=101

@MajesticPotatoe
Copy link
Member

MajesticPotatoe commented Dec 13, 2018

Are you trying to attach a tooltip to a button? Believe the activator goes on the button not the tooltip as such
https://codepen.io/anon/pen/ebJeJG?&editable=true&editors=101

If you have specific questions about how to use Vuetify please check out the following resources:

Thank you for your contribution and interest in improving Vuetify.

@MajesticPotatoe MajesticPotatoe added the invalid The issue is missing information or is not a valid bug/feature request label Dec 13, 2018
@builder7777
Copy link
Author

I'm trying to use it according to the documentation. It says the "Tooltips can wrap any element.", meaning it doesn't have to. I'm trying to implement it the way where it doesn't wrap the button, and it's not working, hence the bug report.

@MajesticPotatoe
Copy link
Member

They can. seems that we don't have an example of a detached, I will add one to the docs for the next update. You can do detached using the attach prop - https://codepen.io/anon/pen/KbzWLb

@MajesticPotatoe MajesticPotatoe added T: documentation and removed invalid The issue is missing information or is not a valid bug/feature request labels Dec 14, 2018
@MajesticPotatoe
Copy link
Member

Updated 9e89ec2

@KaelWD
Copy link
Member

KaelWD commented Dec 27, 2018

This is a bug, see #5860

It only works for v-menu if the value bound to activator changes after mount, and v-tooltip doesn't have any handling for it at all.

That example is also incorrect, attach should only be used to work around scrolling issues.

@zajca
Copy link

zajca commented Feb 8, 2019

I run into same issue, I want to add tooltip to svg path element, and I'm using id selector activator for this.
Here is example: https://codepen.io/anon/pen/rPpPVx?editors=1010

@MajesticPotatoe MajesticPotatoe added the C: VTooltip VTooltip label Apr 9, 2019
@KaelWD KaelWD added the C: VMenu VMenu label Jul 15, 2019
@KaelWD KaelWD changed the title [Bug Report] v-tooltip does not show up with external activator [Bug Report] v-tooltip and v-menu do not work with an external activator Jul 15, 2019
@jacekkarczmarczyk jacekkarczmarczyk added the S: has PR The issue has a pending Pull Request label Aug 1, 2019
@jacekkarczmarczyk jacekkarczmarczyk modified the milestones: v2.x.x, v2.0.x Aug 1, 2019
johnleider added a commit that referenced this issue Aug 12, 2019
fixes #7944
fixes #5890
fixes #5860

* fix(activatable): add listeners for custom activators

* chore(activatable): clean-up

* chore: update packages/vuetify/src/mixins/activatable/index.ts

Co-Authored-By: Dmitry Sharshakov <d3dx12.xx@gmail.com>

* refactor(activatable): switch to add/remove event listener binding
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C: VMenu VMenu C: VTooltip VTooltip S: has PR The issue has a pending Pull Request T: bug Functionality that does not work as intended/expected
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants