Beta feedback: Settings UI & other polish issues (0.4.x)

[BobSilent]

Using 0.4.1 release
On Android 17

• When opening the settings the top most entry is hidden by the Title pyLoad - Preferences, in this picture I am not able to set the url:

Workaround

In wide screen (rotating screen 90 degrees),the Address option gets visible

[UserX404]

I also tested this on my phone and want to share my findings

• Status bar changes icon color to dark and needs default white coloring when device is set to dark theme in system settings
• 3-dot-menu entry "Settings" should be re-phased to "App settings" to let better differentiate between server settings
• inside app settings the scrolable pane/area is hidden behind the bottom navigation bar and the top pyload-settings-banner
• actions on packages or sub-packages are none functional -> queue view - package - long click (delete, move, restart)

@ me if you need device details.

[GammaC0de]

Status bar changes icon color to dark and needs default white coloring when device is set to dark theme in system settings

Please explain what that means

[UserX404]

@GammaC0de
See the top system status bar in the screenshot above. Black notification icons, wifi and cellular signal symbols.
They were changed to black when i open the pyload app.

For reference the expected style (pyload with firefox, etc.)

or the screenshot i posted in #34

[GammaC0de]

actions on packages or sub-packages are none functional -> queue view - package - long click (delete, move, restart)

Explain this, for me it works.

Any other issues before i release a new version?
app-debug.zip

[BobSilent]

@GammaC0de I think I know what @UserX404 means. Compare these two pictures, showing exactly the same in light and dark theme (with released version)

In light you see the status var icons in dark not.

When using other apps, the status bar icons turn are white, e.g. here with the GitHub app

[GammaC0de]

@GammaC0de I think I know what @UserX404 means. Compare these two pictures, showing exactly the same in light and dark theme

This should have been fixed in 3a91bb7 or isn't is?

[BobSilent]

Let me test your provided package here, it was the screenshot from 0.42

Any other issues before i release a new version?

For me just minor things, switch the theme too often crashes the app, but hey this is not a real thing I do offen

I do not like the scrolling titles in the queue, especially of the name geht's to long, in most cases they show me the last parts. I would prefer to not scroll them and maybe make the font size smaller. Better to support some when and edit of the package.

Or I would prefer a Multiline more than scrolling, more like the issues list in the GitHub app, also from font size

[BobSilent]

Yes with the latest provided apk it works, still after two theme changes the app crashes but ok.

[GammaC0de]

still after two theme changes the app crashes but ok.

Can you post the crash log /exception (connect with ADB and and check the Logcat)?

[BobSilent]

Sure I can, but not now this evening or tomorrow, if that's ok. I can also create an own issue for this if you want.

[UserX404]

Thank you for your polishing efforts. Nice to see the app get some love and progress.

Just an idea but can you also implement to respect the systems text size preference?
Setting could be found within accessability settings - Display

[GammaC0de]

can you also implement to respect the systems text size preference?

Isn't that how it works already? from my tests (my phone) the system settings does effect the text size in the app

[UserX404]

Oh, sorry. I checked again and it works indeed but it seems the text size of the scrolling package name is one tick to big.
I reference to the latest fdroid build. So if you've already put some size change in your testbuild please tell me to what version to test.
Thx

Edit:
Just noticed @BobSilent has also already asked for text size reduction

[GammaC0de]

So if you've already put some size change in your testbuild please tell me to what version to test.

check the version here #48 (comment)

[UserX404]

Just uninstalled and reinstalled the mentioned build. The size of the package name text is as big as before.
I would expect a consistent letter size, same as in app-settings menu.
Also the server-setting items are to big and waste too much screen space. They seems to use the same style as the package name.

I saw you changed the menu entry to app-settings. But inside the menu it's "pyload-settings"
Also the menu Server settings: inside the menu it's called "pyload - serversettings"
My localization is set to german, so it may be not the correct spelling or punctuation.
I'd suggest to remove the "pyload-" part and use the same string resources. :-)

[BobSilent]

I am using the system with Font Size and Display Size set to smallest.
In my screenshots one can see that the Package Name is quite big in size even though i use smallest settings, when changing the scaling setting to default this gets even bigger.
You can see the differences between the Github Issues screenshot text sizes and the pyload app screenshot.

So if i could whish me something, I would like prefer something which is more lightweight, so not the big text like now, maybe same font size like the DL size / Package Size, but for the Package name in bold. Maybe also without the small line between the packages, just a little more space, Similar to what the Github Issues screenshot looks like in same comment above.

[UserX404]

@GammaC0de
#52 (comment)

• Font size topic fixed for server settings and package names. Looks much better now
• menu names fixed
  Thank you :-)

[BobSilent]

@GammaC0de maybe just a little thing
When opening a package the single files take up quite a lot of space maybe this can be more condensed.
Also the icon on the left is showing in the middle of the line, where as all text is aligned to top.
And the holster is for me a little bit too much right without space to then right end. Especially compared with the package line above

[BobSilent]

@GammaC0de would you mind sharing a new build for testing?

[GammaC0de]

app-debug.zip

[UserX404]

I think most if the OP issues has been fixed.

But there is still one point left.
When checking server settings -> Download and rotate to landscape some space gets added at the top and the scrollable area is moved down.
When rotating back to portrait the space survives until i back out to server settings again.

When i start from landscape it looks ok but the extra space gets added when rotating to portrait.

[GammaC0de]

But there is still one point left.
When checking server settings -> Download and rotate to landscape some space gets added at the top and the scrollable area is moved down.
When rotating back to portrait the space survives until i back out to server settings again.

Please test:
https://github.com/pyload/pyload-android/releases/tag/v0.4.x%CE%B2

[BobSilent]

@GammaC0de for me the top menu is again to high on top, and behind the status bar on top

[GammaC0de]

for me the top menu is again to high on top, and behind the status bar on top

updated the realease, please try now

[UserX404]

But there is still one point left.
When checking server settings -> Download and rotate to landscape some space gets added at the top and the scrollable area is moved down.
When rotating back to portrait the space survives until i back out to server settings again.

Please test:
https://github.com/pyload/pyload-android/releases/tag/v0.4.x%CE%B2

Now the text is again behind the banner a the setting lines start at the top of the sceen and not below the banner.

[GammaC0de]

updated, try now

[BobSilent]

Yes, thanks looks better now.

Will continue testing...

[UserX404]

Also a 👍 from me.
Working and rotating without issues.

[GammaC0de]

Added multi-server support bd5ed3a
Please test.