[Visibility] Wrong logic for reverse events?

[soar]

Quick explanation

Steps

1. Create some elements for testing:

   ...
   <div id="element1" class="test-element"></div>
   <div id="element2" class="test-element"></div>
   <div id="element3" class="test-element"></div>
   ...

2. Use visibility module on them:

   $('.test-element').each(function () {
   	$(this).visibility({
   		once: false,
   		onTopVisible: function () {
   			console.log("onTopVisible for element", $(this).attr('id'))
   		},
   		onTopVisibleReverse: function () {
   			console.log("onTopVisibleReverse for element", $(this).attr('id'))
   		}
   	});
   });

Expected Result

In Semantic-UI 2.2.10 it works as expected: reverse event was fired only when element goes back to same edge, where it was appeared.

I think, if event is named reverse it should be fired only when element goes back. At this moment it works as onTopInvisible.

Actual Result

reverse event is fired even if element goes through the screen and goes from another edge.

Version

2.2.10 - works as expected
2.3.3 - looks like broken

Testcase

https://jsfiddle.net/soarx/ugv49aLm/

[y0hami]

I think this is just down to some bad naming rather than the events being wrong. Maybe the event names should be changed to better represent why/when they are called.