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

Set a software layer type on ManeuverView #513

Closed
wants to merge 1 commit into from

Conversation

BoD
Copy link

@BoD BoD commented Nov 17, 2017

When using ManeuverView, currently we must do
android:layerType="software" in xml, otherwise the icons will be blurry. This will be confusing for users as this is unusual to impose this (I wasted at least one hour because of this myself).

This change makes the call directly in the Java code so, it is no longer necessary to do it in xml.

@danesfeder
Copy link
Contributor

@BoD Great idea - I actually ran into the same issue when I first implemented the ManeuverView - should have thought of this for developers that wanted to use it as well.

I will create a PR with the changes so CI will fire properly. Thanks for pointing this out!

@danesfeder
Copy link
Contributor

Closing in favor of #514 (so our CI will fire) -- thanks again for pointing this our @BoD and contributing the change

@danesfeder danesfeder closed this Nov 17, 2017
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.

2 participants