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

[API] TransformPoint/Distance additions to Drawing Context #88

Merged
merged 1 commit into from
Apr 3, 2012
Merged

[API] TransformPoint/Distance additions to Drawing Context #88

merged 1 commit into from
Apr 3, 2012

Conversation

hwthomas
Copy link
Contributor

Proposed in Issue #76. This is an initial implementation, with
a visual test added to the Samples/DrawingTransforms example.
Mac Backend is currently NotImplemented.

Proposed in Issue #76. This is an initial implementation, with
a visual test added to the Samples/DrawingTransforms example.
Mac Backend is currently NotImplemented.
@lytico
Copy link
Contributor

lytico commented Mar 29, 2012

would a Matrix (like System.Drawing.Drawing2D.Matrix) be helpfull for you?

@hwthomas
Copy link
Contributor Author

It would mean that Xwt would have to support that struct/class and the associated transforms, and the matrix returned by each backend is not in the exact same format (Cairo layout is the transpose of the GDI+ one - I'm not sure which way round the Mac NSAffineTransform is). My feeling was that this proposal provided the basic operations (which are required, for example, to get the bounds of text that is being displayed with a rotation and translation) without too much API bloat. But I'm very open to suggestions!

@lytico
Copy link
Contributor

lytico commented Mar 29, 2012

ok, I see, that Matrix should have a backend, but working without backend too, be standalone (without canvas, but with backend)
and, with a property, reflect the canvas-backend transforms, like Canvas.Matrix {get}, and maybe Canvas.Matrix {set} too.

slluis added a commit that referenced this pull request Apr 3, 2012
[API] TransformPoint/Distance additions to Drawing Context
@slluis slluis merged commit 40e9a6e into mono:master Apr 3, 2012
@slluis
Copy link
Member

slluis commented Apr 3, 2012

The patch looks great. It's now merged. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants