I think Line/Bar.prototype.onHoverOut is incorrect

[denhartog]

When using option "hideHover: true/'auto', the legend is still displayed after my mouse leaves the the chart area.

Line/Bar.prototype.onHoverOut is currently:
=== 'auto'
and should be
!== false

[oesmith]

When hideHover is true, the legend should not be displayed at all, hence there's no need to hide it.

If the legend is on-screen at all when hideHover == true, then we have a different problem...

[denhartog]

Not according to: http://www.oesmith.co.uk/morris.js/lines.html

hideHover: true/'auto' shows the legend when hovering over it; hideHover: 'always' NEVER displays the legend

Which, except for the error I identified, is how the hideHover command works.

And while we are on the topic, it makes more sense to me to have the options for 'hideHover' be:
false = ALWAYS show;
true = NEVER show;
'auto' = show/hide on mouse in/out

All this being said, I really like Morris and am excited to contribute to this project further.

[oesmith]

The false, true/'auto' and 'always' options are a concession to backwards-compatibility, hence why they're a bit odd.

You're correct that the behaviour doesn't match the docs, though. I've left a comment on the pull request. If you can fix that, I'll get it merged ASAP.

Thanks for the contribution 👍