Skip to content

Add hover / blur callback events #2 #275

Merged
merged 1 commit into from Nov 23, 2012

2 participants

@Peinkillar

merged hover and blur events

see #262

@fnagel
Owner
fnagel commented Oct 25, 2012

Is this the updated PR for #262 ?

@Peinkillar

Yes it is, big thanks for merging it!

@fnagel
Owner
fnagel commented on ca64b0d Nov 10, 2012

Fix seems nice so far, could be merged. Only last question: why do you add optionValue to the event?

@Peinkillar

Hi Felix,
I need the current hovered value to know which element has been affected. Is there another way to get the optionValue without passing it. At the moment a get it in my hover callback by ... e.originalEvent.optionValue ...
Otherwise the only element you can access is the selected one or am I wrong?

@fnagel
Owner
fnagel commented Nov 23, 2012

Ahh ok, seems legit. Merged. Thanks for your contribution!

@fnagel fnagel merged commit 066bb3d into fnagel:selectmenu Nov 23, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.