Bug 531188 - Synchronization decorator is messing with zoom functionality
Summary: Synchronization decorator is messing with zoom functionality
Status: CLOSED FIXED
Alias: None
Product: Sirius
Classification: Modeling
Component: Diagram (show other bugs)
Version: 5.1.0   Edit
Hardware: PC Windows 10
: P3 normal (vote)
Target Milestone: 5.1.2   Edit
Assignee: Pierre Guilet CLA
QA Contact:
URL:
Whiteboard:
Keywords: triaged
Depends on: 530051
Blocks:
  Show dependency tree
 
Reported: 2018-02-15 04:19 EST by Pierre Guilet CLA
Modified: 2018-06-28 12:11 EDT (History)
3 users (show)

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 Pierre Guilet CLA 2018-02-15 04:19:47 EST 
+++ This bug was initially created as a clone of Bug #530051 +++

Doing a page zoom while the synchronization decorator is visible will impact the zoom result badly. The decorator will be taken in consideration when computing the resulting viewport so if the decorator is below the lower diagram element, the zoom will be done to see this decorator.

Steps to reproduce:
1- Import the attached project
2- open the representation 
3- Zoom to 25
4- Use the Page zoom in the tabbar
KO the diagram element  is not near the editor's bottom. This is the synchronization decorator that is.
Comment 1 Eclipse Genie CLA 2018-02-15 04:27:08 EST 
New Gerrit change created: https://git.eclipse.org/r/117417
Comment 3 Pierre Guilet CLA 2018-05-31 04:46:15 EDT 
Verified with OD CE 11 RC2

The zoom button to use for Guillaume scenario is the one in the tabbar.
Comment 4 Pierre Guilet CLA 2018-05-31 05:42:21 EDT 
A new scenario does not work on OD CE 10.1.2.
See the fix for Sirius 6.0.0 https://git.eclipse.org/r/#/c/123322/
Comment 5 Pierre Guilet CLA 2018-05-31 05:42:59 EDT 
The scenario is: https://bugs.eclipse.org/bugs/show_bug.cgi?id=530051#c4
Comment 6 Pierre Guilet CLA 2018-05-31 08:55:19 EDT 
Known bug regarding the zoom with synchronized decorator that will be fixed in 5.1.3:
https://bugs.eclipse.org/bugs/show_bug.cgi?id=535390
Comment 7 Laurent Redor CLA 2018-06-28 12:11:05 EDT 
Available in Sirius 5.1.2, see https://wiki.eclipse.org/Sirius/5.1.2 for details