[OpenLayers-Trac] Re: [OpenLayers] #2891: redrawing a Vector feature does not remove old backgroundGraphic

OpenLayers trac-20090302 at openlayers.org
Tue Oct 19 03:31:35 EDT 2010


#2891: redrawing a Vector feature does not remove old backgroundGraphic
-------------------------------+--------------------------------------------
 Reporter:  ahtih              |       Owner:  ahocevar       
     Type:  bug                |      Status:  new            
 Priority:  minor              |   Milestone:  2.11 Release   
Component:  Renderer.Elements  |     Version:  2.10           
 Keywords:  backgroundGraphic  |       State:  Needs More Work
-------------------------------+--------------------------------------------
Changes (by ahocevar):

  * state:  Review => Needs More Work


Comment:

 Thanks for the patch ahtih. I think this should be simplified a bit. In
 Renderer.Elements::drawGeometry, a few lines below your patch, we check
 for {{{rendered == false}}} and have basically the same code for removing
 the background node again. It would be great if we had the
 {{{document.getElementById}}} for the background node only once.

 Also, can you please add a test case to tests/Renderer/Elements.html that
 proves the removal?

-- 
Ticket URL: <http://trac.openlayers.org/ticket/2891#comment:1>
OpenLayers <http://openlayers.org/>
A free AJAX map viewer


More information about the Trac mailing list