Adds 'hoverable' trigger: hovered tooltip stays put #78

Open
wants to merge 3 commits into
from

Conversation

Projects
None yet
5 participants
@henrik

henrik commented Sep 15, 2011

Useful for tooltips containing links or other stuff to interact with. I'm sure you can think of a more elegant solution, but this appears to work well.

@pokonski

This comment has been minimized.

Show comment
Hide comment
@pokonski

pokonski Sep 15, 2011

I think this could be a separate options key to avoid confusion with "hover".

I think this could be a separate options key to avoid confusion with "hover".

@henrik

This comment has been minimized.

Show comment
Hide comment
@henrik

henrik Sep 15, 2011

@pokonski Hm, that might be a good idea. Or a more different options value, but it's hard to come up with something.

henrik commented Sep 15, 2011

@pokonski Hm, that might be a good idea. Or a more different options value, but it's hard to come up with something.

@pokonski

This comment has been minimized.

Show comment
Hide comment
@pokonski

pokonski Sep 15, 2011

Contributor

Maybe still hoverable, but in something like that

$('#hoverable-example').tipsy({html: true, trigger:'hover', hoverable: true});

because some people might want to use this feature with trigger: 'focus' or any other triggers ;)

Contributor

pokonski commented Sep 15, 2011

Maybe still hoverable, but in something like that

$('#hoverable-example').tipsy({html: true, trigger:'hover', hoverable: true});

because some people might want to use this feature with trigger: 'focus' or any other triggers ;)

@henrik

This comment has been minimized.

Show comment
Hide comment
@henrik

henrik Sep 15, 2011

@pokonski Good idea. With 'focus', it was already hoverable, of course, but not clickable. Added another commit where hoverable is an option of its own and works with 'focus'.

henrik commented Sep 15, 2011

@pokonski Good idea. With 'focus', it was already hoverable, of course, but not clickable. Added another commit where hoverable is an option of its own and works with 'focus'.

@pokonski

This comment has been minimized.

Show comment
Hide comment
@pokonski

pokonski Sep 15, 2011

Contributor

Great, so we continue to wait.

Contributor

pokonski commented Sep 15, 2011

Great, so we continue to wait.

@Roendal

This comment has been minimized.

Show comment
Hide comment
@Roendal

Roendal Nov 7, 2011

Nice feature!

I was looking for something like this. Using henrik's version till it's merged.

Thanks!

Roendal commented Nov 7, 2011

Nice feature!

I was looking for something like this. Using henrik's version till it's merged.

Thanks!

@pklauzinski

This comment has been minimized.

Show comment
Hide comment
@pklauzinski

pklauzinski Aug 8, 2012

Any plans to add this feature in master?

Any plans to add this feature in master?

@pklauzinski

This comment has been minimized.

Show comment
Hide comment
@pklauzinski

pklauzinski Aug 8, 2012

Line 31 - This breaks in Firefox because tipOver and tipOut referenced methods are not yet defined. Apparently Firefox is strict about this. To fix, move tipOver and tipOut definitions above line 30.

Line 31 - This breaks in Firefox because tipOver and tipOut referenced methods are not yet defined. Apparently Firefox is strict about this. To fix, move tipOver and tipOut definitions above line 30.

This comment has been minimized.

Show comment
Hide comment
@ot-eric-vaughan

ot-eric-vaughan May 13, 2014

Thanks! This was a big help.

Thanks! This was a big help.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment