geometer / FBReaderJ

Official FBReaderJ project repository
http://www.fbreader.org/FBReaderJ/
1.83k stars 802 forks source link

[Feature request/Issue] UI design improvements #396

Open easyaspi314 opened 8 years ago

easyaspi314 commented 8 years ago

FBReader is a great e-reader app, but its UI is a bit confusing. @geometer

These are many potential improvements that would help FBReader's UI, mainly in wording.

Preferences

Samsung Galaxy S III SGH-I747M 4.4.2 KitKat official, rooted FBReader free 2.6.10 (2061022)

geometer commented 8 years ago

Thanks, very good request and nice images. We definitely will do this sooner or later. Could you please comment on "small things":

The page transition is jumpy when you don't have your finger on it. Is the book rendered on the main thread, or not rendered by the GPU?

I expect it is slightly better in 2.6.11. Could you please check?

Most settings (fonts, margins, etc) are not applied automatically.

Sorry, what you mean exactly?

Units don't appear to scale to density pixels. (hdpi is 1.5x, xhdpi 2x, etc).

Do you mean font size, margins, etc.? This is a problem: we do not want to change the values in an update for existing users.

easyaspi314 commented 8 years ago

Well the latest update fixed the jumpiness. Much smoother! :D

See #397 for more on the settings not applying.

You just divide the value by the screen density on first run. Is there a preference that contains the previous version? On Mar 24, 2016 7:25 PM, "Nikolay Pultsin" notifications@github.com wrote:

Thanks, very good request and nice images. We definitely will do this sooner or later. Could you please comment on "small things":

The page transition is jumpy when you don't have your finger on it. Is the book rendered on the main thread, or not rendered by the GPU? I expect it is slightly better in 2.6.11. Could you please check?

Most settings (fonts, margins, etc) are not applied automatically. Sorry, what you mean exactly?

Units don't appear to scale to density pixels. (hdpi is 1.5x, xhdpi 2x, etc). Do you mean font size, margins, etc.? This is a problem: we do not want to change the values in an update for existing users.

— You are receiving this because you authored the thread. Reply to this email directly or view it on GitHub https://github.com/geometer/FBReaderJ/issues/396#issuecomment-201071552