Ticket #965 (closed feature: fixed)

Opened 6 years ago

Last modified 5 years ago

rendered geometries blink sometimes in the VML renderer

Reported by: pgiraud Owned by:
Priority: minor Milestone: 2.6 Release
Component: Renderer.VML Version: 2.4
Keywords: Cc: penyaskito@…
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 Download (3.5 KB) - added by pgiraud 6 years ago.
current way to draw point in the vml renderer
create-vml-validdoctype-position-absolute-size-relative.html Download (3.1 KB) - added by pgiraud 6 years ago.
file name self explaining
create-vml-validdoctype-position-absolute-size-absolute.html Download (3.1 KB) - added by pgiraud 6 years ago.
file name self explaining
create-vml-nodoctype-position-absolute-size-absolute.html Download (3.2 KB) - added by pgiraud 6 years ago.
file name self explaining
vml-renderer.patch Download (1.1 KB) - added by pgiraud 6 years ago.
vml-renderer.2.patch Download (1.0 KB) - added by fredj 5 years ago.
same patch

Change History

Changed 6 years ago by pgiraud

Changed 6 years ago by penyaskito

  • cc penyaskito@… added

Changed 6 years ago 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

Changed 6 years ago by crschmidt

  • milestone set to Future

Changed 6 years ago by pgiraud

current way to draw point in the vml renderer

Changed 6 years ago by pgiraud

file name self explaining

Changed 6 years ago by pgiraud

file name self explaining

Changed 6 years ago by pgiraud

file name self explaining

Changed 6 years ago by pgiraud

Changed 6 years ago 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.

Changed 6 years ago by pgiraud

  • keywords review added

Changed 6 years ago 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 ;)

Changed 5 years ago 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.

Changed 5 years ago 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.

Changed 5 years ago by fredj

same patch

Changed 5 years ago 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.

Changed 5 years ago by fredj

  • state changed from Review to Commit

All examples works with IE6 and IE7

Changed 5 years ago 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)

Note: See TracTickets for help on using tickets.