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

Zoom fix #357

Merged
merged 2 commits into from Jul 7, 2017

Conversation

Projects
None yet
2 participants
@JasonLautzenheiser
Owner

JasonLautzenheiser commented Jul 7, 2017

No description provided.

@JasonLautzenheiser JasonLautzenheiser merged commit f04b453 into master Jul 7, 2017

@JasonLautzenheiser

This comment has been minimized.

Owner

JasonLautzenheiser commented Jul 7, 2017

@andrewschultz I figured out the issue that I was having....it truly was my problem and the maps I was using. I think though I will add a feature request to in some way indicate the margins being used in a map so it's more clear looking at the map itself.

I did slightly modify the code to take out a few calcs, that weren't needed....but nothing life changing.

@JasonLautzenheiser JasonLautzenheiser deleted the zoom-fix branch Jul 7, 2017

@andrewschultz

This comment has been minimized.

Collaborator

andrewschultz commented Jul 7, 2017

Wow, good work! What you found pushed me to look into things like reflection to print out all members of a C# structure (that was the executable I was going to send)...so that panned out very well & will be handy for later debugging.

@JasonLautzenheiser JasonLautzenheiser added this to the 1.6.0.0 milestone Jul 7, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment