Setting canvas height about drawableview

[PaNaVTEC]

The pre draw listener you are using is async, not sync. So you are changing the configuration height after you setted the config. if you move the "drawable view.setConfig" inside the pre draw I think you achieve what you want. Check the code, the setConfig configures some things inside the drawableview.

[Odaym]

I get what you mean by I also tried to set the height to 200 or 50 manually without calculation or anything. I will try again and let you know

[PaNaVTEC]

@Odaym So the problem is that the canvas w/h you are setting is minor than the view w/h?

[Odaym]

Well no the problem is that regardless of what number I give I can still draw on all the height of the device

[PaNaVTEC]

I see, to reproduce it I setted canvas height to 100, my drawable view uses all the height and I can paint in all the height wich is not correct, the correct is to limit the view to the canvas height. I got it?

[Odaym]

Yep, that's it

[PaNaVTEC]

@Odaym I just uploaded a fix in the branch "limit_canvas_size" can you test it and tell me what do you think?

[Odaym]

You'll have to excuse me but I don't know how to do what you're suggesting. I can see the branch of course but I don't know what to do with a library that I usually include by specifying that gradle line that I'm used to. How do I test limit_canvas_size branch? I'm thinking that you maybe include the name of the branch in that line I use to include this library?

I tried using this answer to do it but I couldn't get it to download the repo to the path I was specifying http://stackoverflow.com/questions/13719676/gradle-how-to-clone-a-git-repo-in-a-task

[Odaym]

Alright, I cloned the repo to my computer but I couldn't get it to work as a library under my project

1. Grab the "library" folder from the repo and place it under my project/libs and then ask gradle to compile it in build.gradle and settings.gradle

What I tried and did get to work was:

1. Import the whole project DrawableView as a new project in AS and run it.

And I saw that it's perfectly working to limit the height of the canvas, it's great :) Thank you

Are you going to include this into master and upgrade the maven number so that people can include it still in 1 line as before?

[PaNaVTEC]

Hi, I'm still not happy with the solution and is not working good if you are using zoom, so I re-think and implement in a better way, if this is quite enough for you, see the build.gradle in the sample project how to import a dependency and you can work with this branch. I will fix it asap but I want to do some refactor before.

[Odaym]

Done, works fine when included into the project. I can see it running on my own app from that branch. But I will still wait for when its on master, for a much cleaner include process :P Thanks

[PaNaVTEC]

Released 0.6.0