Datepicker: Remove unnecessary mouseover trigger. Fixes #5816

[rxaviers]

The first commit that introduced the mouseover trigger was 3522a23 --- side note, the short form it has now was due to 74d195e. Apparently, it addressed 3647. Although, this specific change seemed only to be a handy (yet unfortunate) way to avoid repeating code.

The two lines ran by the trigger (in the previous days and still today) are [1] and [2] below. The 1 has no effect, since such element has no ".ui-state-hover" class on creation anyway. The 2 adds ".ui-state-hover" to the target selector. But it gets removed later.

1: $(this).parents(".ui-datepicker-calendar").find("a").removeClass("ui-state-hover");
2: $(this).addClass("ui-state-hover");

Unless I'm also missing something, we could safely remove the mouseover trigger.

[jzaefferer]

We should either land this soon or close it, since the rewrite won't have this problem anymore. @tjvantoll could you take a look?

[tjvantoll]

I verified the problem still exists and that this fixes it. "ui-state-hover" is still managed appropriately as well. Let's land it.

[tjvantoll]

fwiw here's the test case I used: http://jsfiddle.net/tj_vantoll/NqFG6/

[jzaefferer]

Okay, landed. Thanks TJ.

[Montago]

Deleting the .mouseover() trigger,. broke the Keyboard Navigation feature !!!!