Ticket #1723 (closed bug: fixed)

Opened 5 years ago

Last modified 5 years ago

Vector feature redrawn offscreen stays invisible

Reported by: jstern81 Owned by:
Priority: minor Milestone: 2.7 Release
Component: general Version: 2.7 RC1
Keywords: Cc: jstern81@…
State: Complete

Description

Windows XP Firefox 3

Vector features with external graphics which are redrawn offscreen remain invisible when they are panned back onscreen, and remain so until a zoom change. This issue reared itself when a selected feature was panned offscreen, unselected, and then panned back onscreen.

See an example at:  http://t2gnow.com/ol/offscreen-feature-redraw/example.html Click to select, pan the feature offscreen, click the map to unselect, then pan the map back to where it should have been. Alternatively, pan the feature offscreen, click the 'draw feature' link to force a redraw, and then pan back to where the feature should be.

Attachments

1723-r7971-A0.patch Download (2.3 KB) - added by ahocevar 5 years ago.
1723-r7971-A1.patch Download (2.3 KB) - added by ahocevar 5 years ago.
Removed no longer needed local unrenderedFeatures variable

Change History

Changed 5 years ago by ahocevar

  • state set to Awaiting User Feedback

jstern81: can you please apply the above patch and see if it fixes your issue?

Changed 5 years ago by ahocevar

Changed 5 years ago by ahocevar

  • state changed from Awaiting User Feedback to Review

Ok, this issue is obvious because we do not handle the unrenderedFeatures in the drawFeature method. The above patch fixes this, resulting (as a side effect) in more compact and better readable code. (Relevant) tests still pass in FF2/3 and IE7. Please review.

Changed 5 years ago by jstern81

Cool, works great and code looks nicer as you say. Could also get rid of the now extraneous local unrenderedFeatures in moveTo:

            if(!zoomChanged && coordSysUnchanged) {
                var unrenderedFeatures = {};

Changed 5 years ago by ahocevar

Removed no longer needed local unrenderedFeatures variable

Changed 5 years ago by ahocevar

Thanks jstern81, I missed that one.

Changed 5 years ago by elemoine

  • state changed from Review to Commit

Looks good to me. Please commit.

Changed 5 years ago by ahocevar

  • keywords pullup added
  • state changed from Commit to Pullup

(In [7976]) moved handling of unrenderedFeatures from moveTo to drawFeature. r=elemoine (pullup #1723)

Changed 5 years ago by euzuro

  • keywords pullup removed
  • status changed from new to closed
  • state changed from Pullup to Complete
  • resolution set to fixed

(In [8012]) Batch merge for rc2 of 2.7. 'svn merge -r7967:HEAD from trunk (Closes #1733) (Closes #1489) (Closes #1639) (Closes #1718) (Closes #1723) (Closes #1732) (Closes #1616) (Closes #1722)

Note: See TracTickets for help on using tickets.