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

Prompt message #11

Open
hongnk opened this issue Dec 12, 2018 · 5 comments
Open

Prompt message #11

hongnk opened this issue Dec 12, 2018 · 5 comments

Comments

@hongnk
Copy link

hongnk commented Dec 12, 2018

Suggestion for more user friendly:

  1. Prompt message: a custom text string like "place QR code inside camera view to scan"

  2. Back button for Android: some recent android devices remove navigation buttons, although navigate back still possible with some gesture, but quite confusing, so better for app to provide back button.

@forrestmid
Copy link
Contributor

  1. I think the prompt message should be handled separately from this library, as different people need the library for different things. For example, in my use case I prompt the user to scan a barcode in a particular location on an object. They then have the option to dismiss it or to permanently dismiss it. If you're instead thinking about a 'hint' type of text that is perpetually in the viewfinder, that could be something to implement that might be handy.

  2. I wasn't aware there were physical navigation buttons on any modern devices. I was under the impression that the android devices all display home, back, and app switcher buttons on the bottom of the screen. If this isn't the case I'm not sure how you'd detect if the device was or was not showing the default software back button and I don't really think showing both would be a good idea. In all my emulators it shows the back button digitally on screen.

@hongnk
Copy link
Author

hongnk commented Dec 12, 2018

Oh thanks for review, let's me clarify:

  1. Prompt message should be empty by default, but if a developer needs it, he can set that parameter and it's good to have the text displays somewhere near bottom of screen. It is an instruction for users. I can't think how that message can be done outside of library. I can prompt users to do something before launching plugin, but it is less friendly because they have not seen the camera view to understand what the prompt means.

  2. In Huawei P20 there's no back button, not even on screen back button. User needs to swipe from left to right at bottom to go back. I have users launch the scanner then thought the app hang because no way to go back. It is common now with android material design to have a back arrow at app title bar. I thought a back arrow like iOS version would be great.

@forrestmid
Copy link
Contributor

When I hear prompt I think a popup or dialog, like an alert. A helper text on the screen makes more sense.

I've put the help wanted tag on this. I haven't done much development on this recently and I'm not very familiar with android to build this type of stuff. Feel free to dig in and try to develop it if you're so inclined.

@hongnk
Copy link
Author

hongnk commented Dec 12, 2018

Thanks @forrestmid . I don't develop in android but will try later and submit PR if possible!

@forrestmid
Copy link
Contributor

This could be solved using #13.

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

No branches or pull requests

2 participants