-
Notifications
You must be signed in to change notification settings - Fork 3
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
Common Graphical Translation #57
Comments
I have written a common graphical translation class (or two) see coord_systems. I'm not going to merge this immediately because it is a rather huge change and I'd like some input. |
@Anstow I wasn't a fan of your coordinate systems approach - maybe next time we meet you can convince me. I ripped your transform class out, made a Transformable class which essentially aggregates a transform, and has a pointer to a parent - so is in many ways similar. This all seems to work nicely, and so I'm closing this as an issue. |
Okay, I spoke too soon. What I've done has reintroduced bugs relating to camera movement. I'd really appreciate it if @Anstow could take a look and fix what I'm doing wrong. I'm testing this in http://github.com/tank-dev/demos/tree/Camera Relevant functions are Edit: I've reverted my changes |
So this is pretty much what the coordinate systems does anyway just with slightly different names. The only difference is there is a separate transformation class as a transform shouldn't really know anything about what its transforming. Also sorry it took me such a long time to reply to this I've been busy. Edit: The rest of the bloat is to make it do sensible things when you remove the parent and the like. |
Graphics classes show a lot of repetition from my rather hacky approach to getting them to actually work with SFML. This is leading to errors and a lot more effort than necessary.
The text was updated successfully, but these errors were encountered: