-
Notifications
You must be signed in to change notification settings - Fork 15
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
AppUI View truncated on square aspect ratios #15
Comments
From what I remember this is a bit tricky, I'd rather like to avoid scrollable if possible. It's also a rather poor UX. I'll see if I can think of something. Not super high priority though. (If anyone would want to tackle this, get in touch to discuss ideas) |
Definitely not high priority, that's why I posted the issue, since I'd inevitably forget 🤣 |
I don't know much about VESC development, but happy to look into this. This would be nice on the smaller screen on a Razr as well. |
@outlandnish thanks for the offer! I'm afraid this won't be easy though. It's not much about VESC development, the UI is in QML. I think some experience with QML is desirable while tackling this, otherwise you'll probably be struggling. There's an added complication here that the whole UI needs to be a single file, which is now already 3500 lines long. The change needs to be done in a very clever way to not add much complexity and not duplicate any code, because otherwise it'll end up being a nightmare to maintain. My idea was to alter the layouts depending on the aspect ratio of the display. But just having a quick look now (it's been a while I wrote the UI), it seems like it's not possible in a trivial way. There may be some clever trick to use or this may just not be feasible... |
Screenshot captured on Galaxy Z Fold 6. Having scrollable when the content doesn't fill the screen (if possible with QML) is a potential solution
The text was updated successfully, but these errors were encountered: