moagrius / MapView

(Deprecated, prefer https://github.com/moagrius/TileView) Android widget roughly described as a hybrid between com.google.android.maps.MapView and iOS's CATiledLayer
http://moagrius.github.com/MapView/documentation
69 stars 35 forks source link

setZoom does nothing in the UI #25

Closed Shusshu closed 11 years ago

Shusshu commented 11 years ago

Maybe it needs a refresh ?

I guess it is telling the system it needs to take imgs from that zoom level and that's it?

moagrius commented 11 years ago

this is the last-minute addition i referenced. this method should just calculate the scale needed at the top of that zoom level and call setScale. I haven't debugged it at all yet, but will as soon as I can.

moagrius commented 11 years ago

I'm considering removing this method, and just controlling scale directly - what do you think? Is it likely to be useful?

Shusshu commented 11 years ago

Yup I think it's not needed if it's simply managing the scale

Benoit Billington

On Fri, Mar 22, 2013 at 6:13 PM, moagrius notifications@github.com wrote:

I'm considering removing this method, and just controlling scale directly

  • what do you think? Is it likely to be useful?

— Reply to this email directly or view it on GitHubhttps://github.com/moagrius/MapView/issues/25#issuecomment-15309304 .

moagrius commented 11 years ago

cool, thanks