OpenLayers OpenLayers

Ticket #965 (closed feature: fixed)

Opened 1 year ago

Last modified 10 months ago

rendered geometries blink sometimes in the VML renderer

Reported by: pgiraud Assigned to:
Priority: minor Milestone: 2.6 Release
Component: Renderer.VML Version: 2.4
Keywords: Cc: penyaskito@gmail.com
State: Complete

Description

In IE, if you try to draw a new point in a vector that already contains drawn features, and you put it on the left and/or on the top of existing geometries, the renderer blinks.

A swf video is attached.

Note: in the old vector branch examples, we don't any effect.

Attachments

create-vml-current.html (3.5 kB) - added by pgiraud on 10/12/07 11:16:06.
current way to draw point in the vml renderer
create-vml-validdoctype-position-absolute-size-relative.html (3.1 kB) - added by pgiraud on 10/12/07 11:17:15.
file name self explaining
create-vml-validdoctype-position-absolute-size-absolute.html (3.1 kB) - added by pgiraud on 10/12/07 11:18:08.
file name self explaining
create-vml-nodoctype-position-absolute-size-absolute.html (3.2 kB) - added by pgiraud on 10/12/07 11:18:36.
file name self explaining
vml-renderer.patch (1.1 kB) - added by pgiraud on 10/12/07 11:24:14.
vml-renderer.2.patch (1.0 kB) - added by fredj on 12/18/07 05:54:58.
same patch

Change History

09/12/07 03:50:58 changed by pgiraud

Video can be seen at http://bluecarto.free.fr/OL/blink_point.html

09/22/07 22:31:12 changed by penyaskito

  • cc set to penyaskito@gmail.com.

09/24/07 09:53:43 changed by fredj

After some investigation, I've found that r2959 is the cause of the bug.

Reversing the patch fix the problem but produce the same errors as described in #564

10/10/07 01:17:00 changed by crschmidt

  • milestone set to Future.

10/12/07 11:16:06 changed by pgiraud

  • attachment create-vml-current.html added.

current way to draw point in the vml renderer

10/12/07 11:17:15 changed by pgiraud

  • attachment create-vml-validdoctype-position-absolute-size-relative.html added.

file name self explaining

10/12/07 11:18:08 changed by pgiraud

  • attachment create-vml-validdoctype-position-absolute-size-absolute.html added.

file name self explaining

10/12/07 11:18:36 changed by pgiraud

  • attachment create-vml-nodoctype-position-absolute-size-absolute.html added.

file name self explaining

10/12/07 11:24:14 changed by pgiraud

  • attachment vml-renderer.patch added.

10/12/07 11:29:47 changed by pgiraud

  • milestone changed from Future to 2.6 Release.

Sorry to try to fix a ticket which milestone was "Future" but this was asked by one of my customers and I think I found something interesting.

I prepared some simple html examples to show the different cases. I hope that they are clear enough. A patch is also attached.

10/12/07 11:46:29 changed by pgiraud

  • keywords set to review.

11/21/07 10:15:03 changed by crschmidt

Fixing a ticket in the future milestone is *always* welcome -- it's only pulling them back *without* having done any work or indicating that you're going to do work that's a problem ;)

12/15/07 09:57:40 changed by crschmidt

Pierre:

If you can get someone (Fred?) to confirm that this fixes things in IE for him, please feel free to comment here, and then it can go in: I can't do a visual inspection, since I don't have IE handy.

12/15/07 11:33:43 changed by crschmidt

  • state changed from Review to Needs More Work.

Bumping from review until I hear back from someone on this: feel free to bump back to review if the answer is yes, or just commit it.

12/18/07 05:54:58 changed by fredj

  • attachment vml-renderer.2.patch added.

same patch

12/18/07 06:07:57 changed by fredj

  • state changed from Needs More Work to Review.

I confirm that the patch fix this issue without regression (#564).

I'll do some manual testing and commit I no objection.

12/18/07 07:53:31 changed by fredj

  • state changed from Review to Commit.

All examples works with IE6 and IE7

12/18/07 08:08:36 changed by fredj

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

(In [5488]) Remove the 'blinking' phenomenon on IE by setting the display elements to 'absolute'. Thanks to pgiraud for the patch (Closes #965)