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

Landscape view not usable on some devices #212

Open
callegar opened this issue Feb 26, 2018 · 5 comments
Open

Landscape view not usable on some devices #212

callegar opened this issue Feb 26, 2018 · 5 comments

Comments

@callegar
Copy link

On my Samsung S5 mini, the rotated, landscape view has become unusable on recent versions of the application. This is because the card title is too tall, not leaving enough space to the barcode, that ends up as large as the screen width, but only a couple of mm tall, which is insufficient for reliable scanning.

@brarcher
Copy link
Owner

I recently created a change which reduces the size of the header for the title, but find that I did not merge it in yet. That change is #213. I took a look at that change with the emulator for small screen sizes, and think that it improves the 1D barcode display.

Attached is a build with that change:
loyalty-card-locker.apk.zip

Would you mind trying it out and seeing if it makes the 1D barcodes usable on your device and provide feedback? If you still see an issue, could you post a screen capture of a barcode so I can see what you are observing? You need not post a screen shot of a real barcode, you can make up and entry.

@shapirus
Copy link

I've had this issue on my Huawei Ascend P6S also. I tried to install the APK you provided above (for some reason it refused to install as an upgrade on top of the version installed by f-droid, so I had to remove, install and restore data from backup), and it does improve things. The barcode height in landscape view looks usable now, please see the attached image (the ruler scale is centimeters). It might still be increased a little further, I think.

loyaltycard

@callegar
Copy link
Author

Same sensations as those from shapirus here. Also IMHO, the height of the barcode in landscape mode could be increased more. My feeling is that in landscape the barcode should really take at least half of the screen height, leaving about 1/3 for the card name and description on top and 1/6 for the stuff encoded on the barcode at the bottom.

@brarcher
Copy link
Owner

brarcher commented Mar 1, 2018

for some reason it refused to install as an upgrade on top of the version installed by f-droid

This is because I do not have the signing key that the F-Droid project uses. The key I sign with is the same key as is used for the APK available on the releases page and on Google Play.

Thanks for the feedback on the landscape layout. I'll push the changes that were in the APK mentioned above into a release. After, I'll look into further improving the landscape layout.

@clach04
Copy link

clach04 commented Jan 18, 2020

If ssue #262 is implemented it may help with this.

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

No branches or pull requests

4 participants