Closed Bug 970225 Opened 7 years ago Closed 7 years ago

[Download Manager] Confirmation screen is displayed once for each selected file

Categories

(Firefox OS Graveyard :: Gaia::Settings, defect)

Other
Other
defect
Not set
normal

Tracking

(blocking-b2g:1.4+, b2g-v1.4 fixed)

VERIFIED FIXED
1.4 S2 (28feb)
blocking-b2g 1.4+
Tracking Status
b2g-v1.4 --- fixed

People

(Reporter: rafael.marquez, Assigned: crdlc)

References

Details

(Whiteboard: [systemsfe])

Attachments

(1 file)

*Procedure
1. Opening Settings app
2. Opening Donwloads list (There are 2 or more files in the list)
3. Press Edit button
4. Press select all button 
5. Press Delete button

*Actual Result
Confirmation screen is displayed once for each selected file. If we have 100 files, we must confirm the removal 100 times

*Expect Result
If you select 2 or more files. Only a confirmation screen appears.

This case is not defined by UX in "DM_settingsDrawerv0.14.pdf". Please can you check this case and update the document.
Flags: needinfo?(fdjabri)
Whiteboard: [systemsfe]
We are going to work on it as soon as we have the wireframe, gracias Rafa
blocking-b2g: --- → 1.4?
We definitely can't ship download manager without this fixed.

Francis will provide an update to clarify the expected behavior.  
Just a note, while we're blocking on this, we decided not to risk 1.4 if download manager and the child bugs aren't ready in time.  We'll evaluate that in the final sprint.
blocking-b2g: 1.4? → 1.4+
In the case of multiple files, a single confirmation dialog should be shown. 

Instead of showing the text: "Delete <filename>?", the confirmation should display: "Delete <n> files?", where n is the number of files selected. I'll update the spec.
Flags: needinfo?(fdjabri)
Assignee: nobody → crdlc
Status: NEW → ASSIGNED
Attached file 16509.html
Attachment #8379683 - Flags: review?(kaze)
Attachment #8379683 - Flags: review?(francisco.jordano)
Comment on attachment 8379683 [details]
16509.html

LGTM, tested on the phone and working perfectly.

Thanks Cristian!
Attachment #8379683 - Flags: review?(francisco.jordano) → review+
ok, thanks a lot mate
Comment on attachment 8379683 [details]
16509.html

Looks good to me, too. Please check my nit on your PR.
Attachment #8379683 - Flags: review?(kaze) → review+
Target Milestone: --- → 1.4 S2 (28feb)
Merged in master:

https://github.com/mozilla-b2g/gaia/commit/221466ded6a14ca42fd34c1544ec17fa8549fcaa
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Verified in master(1.4) branch
Status: RESOLVED → VERIFIED
Comment on attachment 8379683 [details]
16509.html

r- from the l10n side, we'll need plural support for "delete n messages" for non-English languages for which different plural forms exist above 2.
Attachment #8379683 - Flags: review-
(In reply to Axel Hecht [:Pike] from comment #10)
> Comment on attachment 8379683 [details]
> 16509.html
> 
> r- from the l10n side, we'll need plural support for "delete n messages" for
> non-English languages for which different plural forms exist above 2.

Fill a bug to implement it and please explain there what it is because sadly I don't have idea.

PS: Messages are always 2 or more.
Blocks: 977627
(In reply to Cristian Rodriguez (:crdlc) from comment #11)
> Fill a bug to implement it and please explain there what it is because sadly
> I don't have idea.
> 
> PS: Messages are always 2 or more.

Filed bug 977627
OMG, I missed that. Thanks for the good catch, Pike.
(In reply to Cristian Rodriguez (:crdlc) from comment #11)
> Messages are always 2 or more.

A few locales (e.g. Arabic) have a specific plural rules for 2.
More generally, we should *always* think of a generic plural rules in such cases.
Now I am understand, I didn't know :( Sorry
You need to log in before you can comment on or make changes to this bug.