Ticket #1623 (closed bug: fixed)

Opened 5 years ago

Last modified 5 years ago

geometry bounds should be cleared while using path handler

Reported by: pgiraud Owned by: tschaub
Priority: major Milestone: 2.7 Release
Component: Handler.Path Version: 2.6
Keywords: Cc:
State: Complete

Description

Since r7546, the tests are failing in IE6. In this revision, we introduced code that calls geometry.getBounds(). This way, the geometry bounds property is cached. Therefore, it's not kept up to date when user draws a path with the path handler.

Attachments

patch-1623-r7551-A0.diff Download (1.1 KB) - added by pgiraud 5 years ago.

Change History

Changed 5 years ago by pgiraud

Ok, after a quick investigation, it first seems like the unit test doesn't reflect the reality. In the path handler test, the line is drawn with the following actions :

  • mousedown then mouseup on a location,
  • then mousedown and up on an other location.

The problem is that we didn't trigger any mousemove event.

Changed 5 years ago by pgiraud

Changed 5 years ago by ahocevar

  • state set to Commit

I can confirm that the patch fixes the tests.

Changed 5 years ago by pgiraud

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

(In [7552]) handlers test are not reflecting the reality, we now simulate a click, move then click \n that way geometry bounds are cleared, and the tests don't fail in IE6 anymore (See #1602), r=ahocevar,crschmidt (Closes #1623)

Note: See TracTickets for help on using tickets.