search

ponewheel / android-ponewheel

pOneWheel Android app
MIT License
71 stars 25 forks source link

Small screen display issues #51

Open biell opened 6 years ago

biell commented 6 years ago

I attached a screenshot of the pOnewheel app running on my Samsung Galaxy Alpha (an older phone running 5.0.1). Please notice that the 5 shapings contain wrapped words. Also, look just above at the battery section. Due to wrapped words, the text does not all fit in the box.

If you can detect the screen resolution and choose a smaller font at lower screen sizes, that would be helpful.

This screenshot is taken while running the beta app from the app store.

screenshot_2017-12-29-10-18-45

ebabel commented 6 years ago

@biell Thanks for the screenshot! I think Android has a nice new library for auto-sizing text that I've been meaning to try out.

kwatkins commented 6 years ago

@ebabel ugh ya predict this issue will get lots of hits, took forever to get the previous version sized to make everyone happy. good to hear there is a library that does that.

ebabel commented 6 years ago

After trying this out I realize that we can't use the library as we don't want different font sizes for each button and it doesn't resize across multiple views. I tried calculating how much size we'd need, but that was getting very complex. How about just using a horizontal scrollview? Also, do you like having them as buttons?

screen shot 2018-01-05 at 6 12 19 pm

kwatkins commented 6 years ago

Ahh if horizontal scroll bar will do the sizing then sounds good to me. I'm not attached to buttons, anything that is easy to just touch/select (since this needs to be easily selected while on the move)

On Jan 5, 2018 6:24 PM, "Erik Babel" notifications@github.com wrote:

After trying this out I realize that we can't use the library as we don't want different font sizes for each button and it doesn't resize across multiple views. I tried calculating how much size we'd need, but that was getting very complex. How about just using a horizontal scrollview? Also, do you like having them as buttons?

[image: screen shot 2018-01-05 at 6 12 19 pm] https://user-images.githubusercontent.com/1874627/34635518-b10c0bb6-f245-11e7-8222-83ba97a43e52.png

— You are receiving this because you commented.

Reply to this email directly, view it on GitHub https://github.com/ponewheel/android-ponewheel/issues/51#issuecomment-355716394, or mute the thread https://github.com/notifications/unsubscribe-auth/AA0LrbVs9y-yuDj7IubBqRTS3hYq4B9-ks5tHtl0gaJpZM4RPW5M .

biell commented 6 years ago

A scroll would work, but there are two things which should happen if possible with a scroll:

  1. the window should indicate there is more to the left or right somehow.
  2. The current setting should be visible unless the user has scrolled away from it on purpose.
nadavsavio commented 4 years ago

I see this hasn't been updated in quite some time, but I wanted to chime in with a +1 and a screenshot. This bug actually means I can only read 9 of the 16 cells. This is on a Pixel (original) which I don't think of as an especially small screen.

(Tangentially, notice the voltage is only reading half. I just installed a two-x setup and suspect I have a BMS issue, but pointing that out in case there's an issue in the app itself.)

Screenshot_20200131-115306