Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

klighd.piccolo: important change in the drawing coordinate system application, solves #56 #104

Merged
merged 1 commit into from
Feb 11, 2022

Conversation

sailingKieler
Copy link
Member

@sailingKieler sailingKieler commented Dec 16, 2021

hey guys,

for solving #56 I had to do a quite massive change, essential change is this:

ebbe39b#diff-084c1ba8ab1f3979a86885fe58fde7d8c723cbad0fd28be303ad083236b2967aL624-R670

Everything else is basically corresponding adjustment to keep everything in sync and working.

@sailingKieler sailingKieler added the enhancement New feature or request label Dec 16, 2021
@sailingKieler sailingKieler marked this pull request as ready for review December 16, 2021 23:28
Copy link
Member

@NiklasRentzCAU NiklasRentzCAU left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Solves a definitely interesting problem and looks good in the code. I just tested it with the one application where I use some special zoom technique (OsgiViz with zoom to stay) and there the result was always a little off what is expected by zoom to stay, i.e. the focused element did not stay at the same relative position in the viewport and always shifted to some other position when some clipping is used. So this does not break anything but does also not fix the problem for this use case yet.

…lication, solves #56

* now the local transform of the (main) diagram's clip node is *not* applied anymore if the diagram is clipped, while (for non-clipped diagrams) the transform of the diagram's root KNodeTopNode is supposed to be the identity under all circumstances;
* updated related code portions concerning the zooming, the clipping functionality, the outline, and the magnifier lens
@sailingKieler
Copy link
Member Author

sailingKieler commented Jan 3, 2022

The change has been accepted by the customer.
I would however leave the PR open for a few weeks and wait for possible bug reports, since I don't expect conflicting changes in the codebase (and I would love to have all related changes combined within this PR/commit).

If other changes in the affected code regions are required, this PR should be merged beforehand, of course.

@sailingKieler sailingKieler merged commit c90053f into master Feb 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants