search

zamojski / TowerCollector

The OpenCellID and MLS contributor's app.
Mozilla Public License 2.0
226 stars 27 forks source link

Don't force turning off the map to clear the cache #115

Closed jidanni closed 3 years ago

jidanni commented 3 years ago

All I wanted to do was turn off the map temporarily. What does that have to do with clearing the cache? Why can't the two operations be separated somehow. There already is a clear the cache option at the bottom of the menu.

jidanni commented 3 years ago

Screenshot_20210416-120914.jpg

zamojski commented 3 years ago

What would be the reason to temporarily disable the map?

jidanni commented 3 years ago

Well, let's say we are just a conservative user, who e.g., doesn't usually drive around with his sunroof open even though he has one on his car.

And also let's say we live in an underdeveloped country so don't have unlimited data. So our map cache is rather important to us!

Hard-earned cash equals hard-earned cache!

jidanni commented 3 years ago

Also, let's say we are just curious. We just want to see if turning off the map will make the app run even more wonderfully.

But we are faced with giving up our megabytes (gigabytes? #105) of cache just to try it for a minute. So we chicken out.

Who knows, we might have discovered a bug that we could report here 🎅. Alas we will never know.

zamojski commented 3 years ago

I would rather implement a feature to disable map while connected to mobile data and allow to load it only on WiFi. Regarding the cache size it's currently set to 150 MB, so if you scroll far away it will anyway clear the old view.

jidanni commented 3 years ago

Well whatever you do, please don't glue too many features to the same button, as different users may have different needs.