Skip to content

Commit 0a274ff

Browse files
svbergeremlenovouser
authored andcommitted
Remove cursor on mouseleave (#90)
1 parent e241976 commit 0a274ff

File tree

2 files changed

+25
-0
lines changed

2 files changed

+25
-0
lines changed

src/typeahead/menu.js

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -104,6 +104,7 @@ var Menu = (function() {
104104
onSelectableClick = _.bind(this._onSelectableClick, this);
105105
this.$node.on('click.tt', this.selectors.selectable, onSelectableClick);
106106
this.$node.on('mouseover', this.selectors.selectable, function(){ that.setCursor($(this)) });
107+
this.$node.on('mouseleave', function(){ that._removeCursor(); });
107108

108109
_.each(this.datasets, function(dataset) {
109110
dataset

test/typeahead/menu_spec.js

Lines changed: 24 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -33,6 +33,30 @@ describe('Menu', function() {
3333
});
3434
});
3535

36+
describe('when mouseover event is triggered on a selectable', function() {
37+
it('should call setCursor', function() {
38+
var $selectable;
39+
spyOn(this.view, 'setCursor');
40+
41+
this.view.bind();
42+
$selectable = this.$node.find(www.selectors.selectable).first();
43+
$selectable.trigger('mouseover');
44+
45+
expect(this.view.setCursor).toHaveBeenCalledWith($selectable);
46+
});
47+
});
48+
49+
describe('when mouseleave event is triggered on the menu', function() {
50+
it('should call _removeCursor', function() {
51+
spyOn(this.view, '_removeCursor');
52+
53+
this.view.bind();
54+
this.$node.trigger('mouseleave');
55+
56+
expect(this.view._removeCursor).toHaveBeenCalled();
57+
});
58+
});
59+
3660
describe('when rendered is triggered on a dataset', function() {
3761
it('should add empty class to node if empty', function() {
3862
this.dataset.isEmpty.andReturn(true);

0 commit comments

Comments
 (0)