TileView icon indicating copy to clipboard operation
TileView copied to clipboard

Documentation slight outdated (for quick setup)

Open perrochon opened this issue 5 years ago • 7 comments

Tried to get this running. Looks super nice and promising. A few things I noticed while setting it up.

  1. I needed to set min sdk to 26 (Oreo) to compile. Not a problem, and easily fixable through error messages

  2. Could it be that .defineZoomLevel replaced .defineDetail and .addDetailLevel? readme.md refers to defineDetail in the quick setup , and the wiki uses addDetailLevel. The demo code shows how.

perrochon avatar Nov 22 '18 00:11 perrochon

yeah the javadocs are completely outdated and i haven't gone through all the wiki pages. the demo and readme should be relatively correct, although feel free to fork and PR (or just edit the wiki directly) if you have the bandwidth to correct any typos or misnamed methods. otherwise i'll get to it when i can - i usually work in short bursts every few months.

yeah it's defineZoomLevel(int zoom, Object data) now. https://github.com/moagrius/TileView/blob/master/tileview/src/main/java/com/qozix/tileview/TileView.java

moagrius avatar Nov 22 '18 02:11 moagrius

https://github.com/moagrius/TileView/pull/496 for README.me I haven't found a way to edit the Wiki or do a pull request on Wiki. It's one copy paste from the PR to the Wiki for the same line.

perrochon avatar Nov 24 '18 22:11 perrochon

you should be able to edit the wiki directly, but it's np i can do it - what page/line?

moagrius avatar Nov 30 '18 19:11 moagrius

I also get an error message for min sdk to be 26. How can i fix this to use it with min sdk 19?

Pavle37 avatar Dec 02 '18 18:12 Pavle37

This should be in a different issue thread, but nity demo and library have minSdk of 19. Check other Gradle or manifest settings in your project, or other dependencies

moagrius avatar Dec 02 '18 19:12 moagrius

Also in README.md TileView.Builder().build() does not return the build TileView anymore but void and takes the TileView to build as a paramter.

tva-TIS avatar Jul 19 '19 10:07 tva-TIS

i will fix this as soon as i get some time for this project (weeks away)

moagrius avatar Jul 19 '19 18:07 moagrius