OpenLayers OpenLayers

Ticket #1624 (closed feature: fixed)

Opened 4 months ago

Last modified 2 months ago

cursor "inherit" on a vector feature causes IE to load cursor file "inherit"

Reported by: ahocevar Assigned to:
Priority: blocker Milestone: 2.7 Release
Component: Feature.Vector Version: 2.6
Keywords: Cc:
State: Complete

Description

Obviously VML does not recognize the "inherit" keyword from css. In SVG, this works fine.

The attached patch adds an additional check for the value "inherit" before setting the cursor value.

Attachments

1624-r7551-A0.patch (0.7 kB) - added by ahocevar on 07/21/08 10:02:03.

Change History

07/21/08 10:02:03 changed by ahocevar

  • attachment 1624-r7551-A0.patch added.

07/21/08 10:02:43 changed by ahocevar

  • state set to Review.
  • milestone set to 2.7 Release.

07/21/08 10:07:39 changed by pgiraud

  • state changed from Review to Commit.

If you can confirm that tests pass, I'm ok with this patch. Performances are back !

07/21/08 10:36:53 changed by ahocevar

Tests pass in FF2 and IE6. This issue (thanks pgiraud for spotting it) seems to be the reason for the recent complaints abourt vector performance in IE. I just ran tha manual vector-features-performance.html test, and it is ~8 times faster with this fix.

07/21/08 10:37:44 changed by ahocevar

  • status changed from new to closed.
  • state changed from Commit to Complete.
  • resolution set to fixed.

(In [7554]) don't set the cursor to "inherit" in the VML renderer. r=pgiraud (closes #1624)

09/05/08 02:42:25 changed by euzuro

  • component changed from general to Renderer.VML.

09/12/08 08:02:59 changed by fvanderbiest

  • priority changed from minor to blocker.
  • resolution deleted.
  • status changed from closed to reopened.
  • component changed from Renderer.VML to Feature.Vector.

I reopen this ticket because it happens again on IE with OpenLayers.Feature.Vector.styledefault? and OpenLayers.Feature.Vector.styletemporary? specifying cursor: "inherit".

Replacing "inherit" by "pointer" stops the calls to /inherit on the server when IE is being used (which is pretty weird !)

(follow-up: ↓ 8 ) 09/12/08 08:18:39 changed by ahocevar

  • state changed from Complete to Awaiting User Feedback.

What version of OpenLayers are you using? I just checked in current trunk and the 2.7rc1 branch, and in both the code that prevents this behaviour is still there.

(in reply to: ↑ 7 ) 09/12/08 08:34:11 changed by fvanderbiest

  • status changed from reopened to closed.
  • state changed from Awaiting User Feedback to Complete.
  • resolution set to fixed.

I'm currently stuck at r7479. As it was fixed in r7554, this explains why it happens. Sorry for the "noise".

Replying to ahocevar:

What version of OpenLayers are you using? I just checked in current trunk and the 2.7rc1 branch, and in both the code that prevents this behaviour is still there.