fix(Portal): hover behavior fixed in Portal's event handlers

[Fabianopb]

Originally Portal closes on mouseleave. However, mouseleave is fired when "the pointer has exited the element and all of its descendants" (ref: https://developer.mozilla.org/en-US/docs/Web/Events/mouseleave).

For that reason the Popup was closing when the pointer exited the inner elements, but not necessarily the Portal itself (which is used to render the Popup).

Gif of the fixed behavior:

The solution was to cancel the handlePortalMouseLeave event in case the event's target doesn't match the portal node. A test was also added. It seemed a clean enough solution for this bug, let me know what you think!

Cheers!
(fixes #3239)

[welcome]

💖 Thanks for opening this pull request! 💖

Here is a list of things that will help get it across the finish line:

• Run yarn lint locally to catch formatting errors. This will fix some errors automatically, commit and push any changes.
• Run yarn test locally to catch errors. This ensures all components still behave as they should.
• Run yarn start to run the doc site locally and try a few pages, ensuring everything is in good working order.
• Include tests when adding/changing behavior.

We get a lot of pull requests on this repo, so please be patient and we will get back to you as soon as we can.

[codecov-io]

Codecov Report

Merging #3257 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #3257      +/-   ##
==========================================
+ Coverage   99.92%   99.92%   +<.01%     
==========================================
  Files         169      169              
  Lines        2802     2803       +1     
==========================================
+ Hits         2800     2801       +1     
  Misses          2        2

Impacted Files	Coverage Δ
src/addons/Portal/Portal.js	100% <100%> (ø)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update dfcbdd5...2a348f8. Read the comment docs.

[layershifter]

@Fabianopb thank you for the investigation, fix looks correctly to me. I will test it on weekend.

[levithomason]

Fantastic, thanks! I've updated the branch to the latest and approved it for merge. We can merge once tests pass.

[layershifter]

@levithomason Codecov will never pass if you made "Update branch" on Github:

https://circleci.com/gh/Semantic-Org/Semantic-UI-React/6167

[welcome]

Congrats on merging your first pull request! 🎉🎉🎉

[wangjitai]

Hi there, Is there any idea to resolve please?

should I update the Semantic ui react version? Thanks!

[Fabianopb]

@wangjitai the team has been cutting releases very often and last one was 19 days ago, I'd say it shouldn't take too long for the next one.

[avinashjoshi]

Exactly what i was looking for - Hopefully the new version releases soon!

[flq]

Any idea when this will go to npm?

[ekeren]

Hey, Any idea when this will be released as part as 0.84?