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.

Sign up for GitHub

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

Jump to bottom

Doble Click on Fab buttons #11

Closed
XavierTecBr opened this issue Sep 21, 2017 · 10 comments
Closed

Doble Click on Fab buttons #11

XavierTecBr opened this issue Sep 21, 2017 · 10 comments

Comments

@XavierTecBr
Copy link

hi.
i'm using fab with the tooltip on fabButtons, like issue #10 .

But when i do duble click on Fab, The fabButtons missing but his tooltips not.
@hazzo
Copy link
Contributor

hazzo commented Sep 21, 2017

Hi @jgnovais, how are you triggering the fab action buttons with double click? Some code sample will be good for helping.

@XavierTecBr
Copy link
Author

Yes it is with a double click.
Tomorrow morning i will send a print screen.

@XavierTecBr
Copy link
Author

image

@hazzo
Copy link
Contributor

hazzo commented Sep 22, 2017

Yes I understand what you mean, but how are you implementing double click with our fab? We don't support from the box any other event than simple click...

@XavierTecBr
Copy link
Author

Just click twice on fab, i implemented nothing :D

@hazzo
Copy link
Contributor

hazzo commented Sep 22, 2017

Well that's why it's happening. Consecutive double click has apparently a bug where the tooltips don't have enough time to appear and lose their anchors. If you click once let the buttons appear and then click again this won't happen. I'll investigate deeper about this bug.

@XavierTecBr
Copy link
Author

Thanks

hazzo added a commit that referenced this issue Sep 29, 2017
@hazzo
Fix regarding issue #11
b082d85 
- Evaluate again inside timeout if toggle is false or true to assure
that tooltips are only shown when  actions are displayed.
- Rolledback to v-show and removed else statement on showTooltip mehotd.
- No need to hide tooltips because will be destroyed by v-if when element
is removed from dom.
- Added 100ms for avoiding delay issues and match animation transition
time of 0.7s (maybe more time is needed)
@hazzo
Copy link
Contributor

hazzo commented Sep 29, 2017

Hi @jgnovais,

issue has been fixed on ralease 2.1.1.
Update, try it and please don't doubt on commenting here again.

Thanks for reporting.

@hazzo hazzo closed this as completed Sep 29, 2017
@XavierTecBr
Copy link
Author

\o/ Thanks i will to do this

@XavierTecBr
Copy link
Author

Thanks @hazzo very well 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@hazzo @XavierTecBr