-
-
Notifications
You must be signed in to change notification settings - Fork 156
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
Feature request: Android based watch support #37
Comments
Comment by brarcher Trying to install the app to a watch simulator it says that the app does not declare 'uses-feature watch', and thus will not install. Are you modifying the application to install on your watch, or are forcing the install some other way? I've not considered the use case of using a watch for this application. Supporting a watch requires a separate APK specially for the watch, from what I understand. Supporting a watch is fine, though I do not have one myself so I do not have much incentive. I'd gladly accept contributions which support a watch in a maintainable way. |
Comment by gtr33m I actually just installed the APK directly using adb, and it installed Not sure if the attached photos will post from this email. If not I'll I'm happy to donate, but unfortunately not enough to get you a watch to Regards, Mark Hawkins 20181106_183040.jpg 20181106_183047.jpg 20181106_183057.jpg 20181106_183109.jpg On 6 Nov. 2018 17:01, "Branden Archer" [email protected] wrote: Trying to install the app to a watch simulator it says that the app does I've not considered the use case of using a watch for this application. — |
Comment by brarcher
Hm. Maybe Android Studio prevented me from installing the APK on the simulator, but doing it with adb directly works fine. I expect that the Play store would be similar, in that it would restrict it. I do not think that F-Droid supports watches (link). Probably the only way to do it without modifying the app is to side load it with adb. I tried to access the photos, but they are links to Drive. I requested permission to see the pictures. It may be easier to drag/drop them into the comment box on Github, if that is possible.
lol, that is fine. (: So, asking about the use case, what happens when you are at the grocery store and you need to give the clerk your card? I usually hand over my phone and they scan it. Would you be taking off your watch and giving it to them, trying to reach over and scan the watch, or something else? How would that work? |
Comment by gtr33m I've shared an item with you: 20181106_183057.jpg It's not an attachment – it's stored online. To open this item, just click |
Comment by gtr33m I've shared an item with you: 20181106_183047.jpg It's not an attachment – it's stored online. To open this item, just click |
Comment by gtr33m I've shared an item with you: 20181106_183040.jpg It's not an attachment – it's stored online. To open this item, just click |
Comment by gtr33m I need to clarify that my watch is an Amazfit Stratos, which is Android 5.1 The majority of your users will probably have Wear OS watches, which would On the other hand, the nice thing about being Android 5.1 based is that The only real problem is the size of the header which takes up too much of Regards, Mark On Wed, 7 Nov 2018 at 17:20, Branden Archer [email protected]
-- |
Comment by brarcher Ah, OK. You explanation of what the device helps make more sense. Namely, it is not a Wear OS device after all. Are you able to build the code? If so, could you try the following to see if it better reduces the header: The following two lines in loyalty_card_view_layout.xml control the height of the title bar when displaying a barcode:
If you change them to something else, for example the following:
You should see the title bar decrease in size. On my emulator it went from: to If that gets what you are hoping for, then success! A setting for that field could be created, that way you could configure it instead of it being fixed. Let me know if the suggested change gets you what you need. |
Comment by brarcher Also, do you mind if I put your pictures into the thread for reference? You listed a barcode in one picture, and I want to be sure its value is not sensitive. |
Comment by gtr33m No problems I can modify and build. I'll give it a try on the weekend and Is there a setting that sets the default font size for each text field as Regards, Mark Hawkins On Thu, 8 Nov. 2018, 15:34 Branden Archer <[email protected] wrote:
|
Comment by gtr33m Go ahead. It's my park run barcode, but if anyone wants to use it to record Regards, Mark Hawkins On 8 Nov. 2018 15:38, "Mark Hawkins" [email protected] wrote: No problems I can modify and build. I'll give it a try on the weekend and Is there a setting that sets the default font size for each text field as Regards, Mark Hawkins On Thu, 8 Nov. 2018, 15:34 Branden Archer <[email protected] wrote:
|
Comment by brarcher Here are the pictures you posted: |
Comment by brarcher
The default value for the setting are in settings.xml under |
Comment by alexanderadam For Garmin there's already "Barcode Wallet" but it cannot sync anything with loyalty card locker. |
I'm closing this issue because smartwatches running stock Android are very very very rare. Wear OS support would be more useful. |
I think they are actually relatively common (Amazfit, some Garmins, ...), and the same codebase could perhaps be used for smaller displays and Wear watches |
Oh? Do you have some example devices for me, or a webpage listing this? Would be good to know a bit more if it is indeed more common than I thought. |
Not sure I know all, but:
I guess there are several more which I do not know of, so feel free to correct me |
galaxy watch 4 and also pixel watch which will release soon |
Those two run WearOS, not Android, and thus aren't related to this GitHub issue. This is specifically about Android itself running on smartwatches. Those watches would be better served by #25 |
To my knowledge, the app is pretty good at responding to different UI scales (and a lot of UI has changed since the LKL days). Will need more info on what exact parts don't work well to look into this further. |
Issue by gtr33m
Tuesday Nov 06, 2018 at 03:18 GMT
Originally opened as brarcher/loyalty-card-locker#282
I have an amazfit Stratos which is Android 5.1 based with a 320x300 pixel resolution. The app installs and works with a bit of tweaking of the fonts, however the title bar of an individual barcode is too large and takes up too much of the screen. Could you please include an optimisation for a smaller screen?
Thanks
The text was updated successfully, but these errors were encountered: