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

Make the Volume, mode, and save channel; widgets more usable #26

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

marcusbirkin
Copy link

@marcusbirkin marcusbirkin commented Dec 17, 2019

Rescaled and moved Volume, mode, and save channel; widgets
Particularly in landscape devices, these buttons could be difficult to use on embedded devices

Make the Volume, mode, and save channel; widgets more usable
Make the Volume, mode, and save channel; widgets more usable
@marcusbirkin
Copy link
Author

Landscape:
image
Portrait:
image

@freshollie
Copy link
Owner

Looks good, but does this work with SlideShow images on the screen? The image was already pretty constrained? What does it look like on 7" devices?

@marcusbirkin
Copy link
Author

I've never seen slideshow pictures, but the widget for that remained unchanged.

Screen shot from real hardware, running on Android 8.1 with a 7" screen (n.b. low signal due to no antenna):
monkeyboard

@freshollie
Copy link
Owner

Why does the scaled icon look so much worse on real hardware? Very weird. Yeah no SlideShow in the UK. Maybe this could be something which could be activated in settings?

Scale drawables down, and not up
Scale drawables down, and not up
@marcusbirkin
Copy link
Author

I've increased the size of the icon, and then scaled down when drawn.
That removes the scaling artifacts.

As to why this isn't shown in the simulator, maybe the emulated gfx chip is better than the cheap one in my hardware?

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

Successfully merging this pull request may close these issues.

2 participants