As discussed in bug 883341, there are certain conditions that will cause the contacts pick activity to display an empty dialog box. The contacts code expects that the activity will only be triggered if the type matches one of its filter values. Currently, however, all apps providing the pick activity will be triggered when no type is specified. In that case, contacts does not properly initialize its dialog message. Contacts should have a decent, catch-all fallback message like "No data found".
Nominating since this bug was broken out from bug 883341 which was leo+.
blocking-b2g: --- → leo?
UX, do you have a preference on the default message? Given timeframes for leo and l10n strings we are going to proceed with a basic message.
Created attachment 768955 [details] Link to Github https://github.com/mozilla-b2g/gaia/pull/10700 Please find a link to Github https://github.com/mozilla-b2g/gaia/pull/10700
Attachment #768955 - Flags: review?(arcturus)
Attachment #768955 - Flags: review?(arcturus) → review?(francisco.jordano)
Two new strings that need to be localized in this bug: filetypeUnsupportedTitle = Invalid type of data filetypeUnsupportedMessage = This application cannot handle the type of data requested. Not sure of the process to request l10n support. Adding needinfo? :l10n.
See https://groups.google.com/forum/#!topic/mozilla.dev.gaia/SrFrWrBNl20 for instructions, we should really try to not take this.
UX is still giving priority to blocking bugs. Assigning to Rob for advice on strings while noting that this is late-l10n and not yet a blocker.
Flags: needinfo?(firefoxos-ux-bugzilla) → needinfo?(rmacdonald)
Created attachment 768987 [details] Empty dialog screenshot. Copy screenshot from bug 883341 to help release drivers.
Summary: contacts pick activity can display empty dialog box → contacts pick activity can display empty dialog box when triggered for "any" type such as from <input type="file">
To help triage, note that this bug was derived from bug 883341 which is already leo+.
+1 to Ben, this is the patch for to save contacts to suffer from bug 883341 that is already a leo+. Thanks!
(Don't know why my previous comment removed the needinfo :S) Already made some comments in the code about things we are missing. Also another idea came to my mind: Right now we launch contacts as an activity and load the contacts list, wait in it, and when the user clicks on a contact we display that message. Another alternative is doing this control here: https://github.com/mozilla-b2g/gaia/blob/master/apps/communications/contacts/js/activities.js#L56 sanitise the pick activity info needed, and if we don't have the proper filters display the alert that we are showing in ths PR. With that, the user will get feedback sooner, and wont need to click in a contact to know the contacts app is not going to handle the request. If we decide to go for this option we should still modify the code so the dataPickHandler function is safe anyway. wdyt folks?
Summary: contacts pick activity can display empty dialog box when triggered for "any" type such as from <input type="file"> → contacts pick activity displays empty dialog box when triggered from <input type="file"> or other requests to pick without an associated type
Latest developments on bug 883341 are making this one not really important now. I'm not sure it should be leo+: the fix for 883341 seems to be okay as far as I've tested, it avoid any access to Contacts from the described case, hence the user cannot get trapped in the present bug.
Agree. The fix for bug 883341 looks good and removes the urgency for fixing this. I'll go ahead and remove my leo? nom and the late-l10n flag. Leaving this open, though, since its probably a good idea to do something other than the blank dialog with the unexpected activity type. Now that we have required=true in our filters, perhaps we should immediately postCancel() or even throw()? Thoughts?
blocking-b2g: leo? → ---
Comment on attachment 768955 [details] Link to Github https://github.com/mozilla-b2g/gaia/pull/10700 As the main problem has been fixed in bug 883341, this PR is not needed anymore, but IMHO, we should modify this part in order to not to have any possible case where we display an empty dialog. In this case the wording I think should change. Thanks a lot guys for the work to solve this!
Attachment #768955 - Flags: review?(francisco.jordano) → review-
This hasn't had any activity in the last week. Can we mark this WONTFIX since bug 883341 should prevent this situation from happening?
As per comment 14
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.