1414992 - QR Code App is mysterious

Status: RESOLVED WONTFIX

(Firefox for Android Graveyard :: Settings and Preferences, defect, P5)

Description

[Ben Kero]

User Agent: Mozilla/5.0 (X11; Linux x86_64; rv:58.0) Gecko/20100101 Firefox/58.0
Build ID: 20171031220132

Steps to reproduce:

I attempted to use 'Remote debugging via Wi-Fi' in Firefox Nightly (at present date)


Actual results:

The box was greyed out because I didn't have the exact ZXing Barcode Scanner app installed. I had a clone app that tenuously claimed to be the ZXing app.


Expected results:

The button should have directed me to the Google Play Store (or a direct APK) for the only barcode scanner app (ZXing app) that it supports.

Comment 1

[J. Ryan Stinnett [:jryans] (Use needinfo, replies may be slow)]

Just to add more context on the current state:

Currently there is a "Learn more" link next to this setting that goes to an MDN page.  But it's the page about USB debugging, so first you have to click a link there to go to Wi-Fi debugging.  Then on _that_ page, we do link to the app:

https://play.google.com/store/apps/details?id=com.google.zxing.client.android

If we add a button / link that opens the Play Store on device directly like this bug suggests, that should greatly reduce the chance of confusion.

Comment 2

[Ian Moody [:Kwan] (UTC+0)]

Attachment: Bug 1414992 - Add link to the supported QR reader in developer preferences.

Only the ZXing Barcode Reader app is supported, so link to it directly.

Review commit: https://reviewboard.mozilla.org/r/204832/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/204832/

Comment 3

[Ian Moody [:Kwan] (UTC+0)]

Attachment: Bug 1414992 - Replace wifi debugging pref with the scanner link if not installed.

The pref isn't useful without the scanner so hide it, and use its disabled
summary for the link instead.

Review commit: https://reviewboard.mozilla.org/r/204834/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/204834/

Comment 4

[Ian Moody [:Kwan] (UTC+0)]

Attachment: Bug 1414992 - Make the scanner install link a market:// link.

So the Play Store app opens directly.

Review commit: https://reviewboard.mozilla.org/r/204836/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/204836/

Comment 5

[Ian Moody [:Kwan] (UTC+0)]

Have some rough work implementing this and looking for feedback.

Open questions?
Should it be the market:// link so it immediately opens the Play Store? (3rd patch does this)
Should the wifi pref be replaced by the link? (2nd patch does this)
Should a new summary be written being more explicit about it needing to be a certain app? (not done)

Comment 6

[Nevin Chen(Not active on Bugzilla)]

(In reply to Ian Moody [:Kwan] from comment #5)
> Have some rough work implementing this and looking for feedback.
> 
> Open questions?
> Should it be the market:// link so it immediately opens the Play Store? (3rd patch does this)
This is by design so we would like to keep it as is.

> Should the wifi pref be replaced by the link? (2nd patch does this)
> Should a new summary be written being more explicit about it needing to be a
> certain app? (not done)

I'll ask UX team to give us more suggestion. Thanks a lot!

Comment 7

[Ian Moody [:Kwan] (UTC+0)]

Fennec is dead.