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

Allows fix for issue #19 where Images set are always Template. #20

Closed

Conversation

tysonkerridge
Copy link

Basically a fix for #19 (Request: Allow non-template Images or Access to the ImageView in some way) which allows for an Boolean value, setImagesAsTemplate, to be set on the SVPulsingAnnotationView which determines whether images added to the imageView and headingImageView are to be the exact image that you give it, or rendered as Template images. Without this fix, any images given to these Image Views are set with the UIImageRenderingModeAlwaysTemplate Rendering Mode.

This fix is much better than the subclass "hack" seen in issue #19, with only 4 line changes compared to having to subclass, override two methods, and access the private imageView property through searching the subviews.

The changes don't break anything in the master branch and the new Boolean defaults to YES as to not change anything for those who would update to this commit and still want the Rendering Mode defaulting to Template.

…the imageView and headingImageView to be set as is instead of with the UIImageRenderingModeAlwaysTemplate Rendering Mode. Defaults to YES as to not change anything for those who would update to this commit and still want the Rendering Mode defaulting to Template. Fixes issue TransitApp#19 on SVPulsingAnnotationView.
@tysonkerridge
Copy link
Author

tysonkerridge commented Oct 20, 2016

Closing this as commit 2638b15 fixes the majority of this issue by making the imageView property public. If you needed the headingImageView public as well then the issue may still be viable but using it on headingImageView in the same way might not have been intended.

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.

1 participant