Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Small screen display issues #51

Open
biell opened this issue Dec 29, 2017 · 6 comments
Open

Small screen display issues #51

biell opened this issue Dec 29, 2017 · 6 comments
Labels

Comments

@biell
Copy link
Collaborator

biell commented Dec 29, 2017

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
Copy link
Collaborator

ebabel commented Dec 29, 2017

@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
Copy link
Collaborator

kwatkins commented Jan 5, 2018

@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
Copy link
Collaborator

ebabel commented Jan 6, 2018

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
Copy link
Collaborator

kwatkins commented Jan 6, 2018 via email

@biell
Copy link
Collaborator Author

biell commented Jan 6, 2018

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.

@ebabel ebabel added the bug label Jan 12, 2018
@nadavsavio
Copy link

nadavsavio commented Jan 31, 2020

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants