added mouseover event handler

[jmandel1027]

Hi there,

I added an event handler for mouseover as requested in #3217. I ran the test suite and everything checks out, fairly new to contributing to open source and would love some feedback!

I've been really enjoying working and learning with mapbox on creative projects like printmaking and custom map vis. Thought it would be a fun learning experience to make some pr's! 🎊

Cheers!
-Jason

Launch Checklist

• briefly describe the changes in this PR
• write tests for all new functionality
• document any changes to public APIs
• manually test the debug page
• post benchmark scores

[mourner]

Looks good to me! Maybe we could cut out the target looking part into a separate function to avoid duplication, but it's probably fine as it is.

[jmandel1027]

Thanks @mourner! 🌱 🎉 Yeaa I was thinking about that as well, I decided to borrow it from mousemove so that it would be specified to the desired dom node on default. I can splice that into a separate function on a future issue/pr request though! 👍

[jmandel1027]

Apologies for the noise on this pull request!

I was working on another issue and realized I had merged it to the same branch I was working on.
I went back and reverted that commit to keep things clean on this PR. 😅

[mourner]

No worries! BTW, you can force-push to the branch with the original commit hash instead of reverting.

[jmandel1027]

mmmh good to know! Will deff keep that in mind, thanks!