Bug Tracker

Modify

Ticket #1970 (closed bug: fixed)

Opened 5 years ago

Last modified 5 years ago

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:
Blocking: Blocked by:

Description (last modified by davidserduke) (diff)

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

comment:1 Changed 5 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 5 years ago by davidserduke

  • Owner set to davidserduke
  • Status changed from new to assigned
  • Description modified (diff)

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

comment:3 Changed 5 years ago by davidserduke

  • Status changed from assigned to closed
  • Resolution set to fixed

Fixed in [3951].

Please follow the  bug reporting guidlines and use  jsFiddle when providing test cases and demonstrations instead of pasting the code in the ticket.

View

Add a comment

Modify Ticket

Action
as closed
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.