Bug 236894 - viewport should have atomic zoomTo() method
Summary: viewport should have atomic zoomTo() method
Status: NEW
Alias: None
Product: GEF
Classification: Tools
Component: GEF-Legacy Draw2d (show other bugs)
Version: 3.4   Edit
Hardware: PC Windows XP
: P3 enhancement (vote)
Target Milestone: ---   Edit
Assignee: gef-inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-06-12 11:10 EDT by Marc Gobeil CLA
Modified: 2008-06-12 11:42 EDT (History)
0 users

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Marc Gobeil CLA 2008-06-12 11:10:47 EDT 
Build ID: I20080523-0100

Steps To Reproduce:
1. on a gmf diagram, use Zoom To Fit

More information:
Zoom to fit, or any other function that wants to zoom to arbitrary new bounds has to use separate setZoom() and setLocation() calls, and it causes unnatural animation.

Instead of a direct straight-line shift, objects appear to follow an arc with a bend in it as the animation hands off from scaling to translation.
Comment 1 Marc Gobeil CLA 2008-06-12 11:42:20 EDT 
after taking a closer look, maybe a better signature for the required function would be Viewport.setBounds(Rectangle bounds), which sets all the parameters and then sets off a single property changed event, instead of one every time a single position or extent of its independant x & y RangeModels changes.