aarddict / android

Aard Dictionary for Android
GNU General Public License v3.0
85 stars 43 forks source link

Unable to zoom out #93

Closed smaeul closed 10 years ago

smaeul commented 10 years ago

Using a Nexus 5 / Android 4.4.2 with the latest Aard from F-Droid (1.6.7).

When first installed, all of the text was tiny, (this is understandable, due to the XXHDPI screen) so I needed to zoom in.

However, I am unable to zoom back out, or even out at all; this prevents me from doing any on-page searches, as they will zoom further in.

itkach commented 10 years ago

Indeed, I see this too. I will look into it, thanks for reporting. As a temp workaround, you can blow away app data and rescan device for dictionaries. This will reset the zoom and you'll be able to zoom in to a comfortable level.

itkach commented 10 years ago

Restoring article view scroll position when navigating back also no longer works.

This is related to all sorts of breakage in Android 4.4 WebView, such as

https://code.google.com/p/android/issues/detail?id=62443 https://code.google.com/p/android/issues/detail?id=62378 https://code.google.com/p/android/issues/detail?id=64237 https://code.google.com/p/android/issues/detail?id=63632

I'm leaning to disabling zoom altogether and hope that new TEXT_AUTOSIZING algorithm will do the right thing for all devices. Find in page, however, still results in text size jumping between slightly smaller and slightly bigger each time Next is tapped. Not sure why, perhaps yet another WebView bug.

itkach commented 10 years ago

Fixed in 101fffca622beade86837077a324c929debb517a. The solution was to make viewport non scalable and use text zoom instead of setting scale (on Android 4.x, nothing changed for older versions). Restoring scroll position still doesn't work on Android 4.4 though, I don't know how to fix it.