Bug Tracker

Opened 13 years ago

Closed 13 years ago

#1970 closed bug (fixed)

hover in Opera disables changes in cursor icon

Reported by: nene Owned by: davidserduke
Priority: minor Milestone: 1.2.2
Component: core Version: 1.2.1
Keywords: Cc:
Blocked by: Blocking:

Description (last modified by davidserduke)

When using any hover effect, even an empty one like this:

$(function(){
    $("li").hover( function () {/* nothing */}, function () {/* nothing */} );
});

On HTML like this:

<ul>
<li><a href="#">Hello</a> world!</li>
<li><a href="#">Hello</a> world!</li>
<li><a href="#">Hello</a> world!</li>
</ul>

then, when hovering cursor from non-link content to link, the cursor does not change shape. It should change from simple arrow to pointing hand, but does not.

This happens only in Opera (tried both Windows and Linux versions).

Simple solution: hover() should return true instead of false when moused over a sub-element.

Example fix:

jQuery.fn.extend({
	hover: function(f,g) {
		// A private function for handling mouse 'hovering'
		function handleHover(e) {
			// Check if mouse(over|out) are still within the same parent element
			var p = e.relatedTarget;
			// Traverse up the tree
			while ( p && p != this ) try { p = p.parentNode; } catch(e) { p = this; };
			// If we actually just moused on to a sub-element, ignore it
			if ( p == this ) return true; // this was previously FALSE
			// Execute the right function
			return (e.type == "mouseover" ? f : g).apply(this, [e]);
		}
		// Bind the function to the two event listeners
		return this.mouseover(handleHover).mouseout(handleHover);
	}
});

Change History (3)

comment:1 Changed 13 years ago by nene

Damn... I just tried out the formatting options and mistakenly hit the submit button. The last code should look like this:

jQuery.fn.extend({
	hover: function(f,g) {
		
		// A private function for handling mouse 'hovering'
		function handleHover(e) {
			// Check if mouse(over|out) are still within the same parent element
			var p = e.relatedTarget;
	
			// Traverse up the tree
			while ( p && p != this ) try { p = p.parentNode; } catch(e) { p = this; };
			
			// If we actually just moused on to a sub-element, ignore it
			if ( p == this ) return true; // this was previously FALSE
			
			// Execute the right function
			return (e.type == "mouseover" ? f : g).apply(this, [e]);
		}
		
		// Bind the function to the two event listeners
		return this.mouseover(handleHover).mouseout(handleHover);
	}
});

comment:2 Changed 13 years ago by davidserduke

Description: modified (diff)
Owner: set to davidserduke
Status: newassigned

You fix looks good in my limited tests. I'm trying to think about any other possible consequences.

comment:3 Changed 13 years ago by davidserduke

Resolution: fixed
Status: assignedclosed

Fixed in [3951].

Note: See TracTickets for help on using tickets.