Closed Bug 970225 Opened 11 years ago Closed 11 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)
Status: ASSIGNED → RESOLVED
Closed: 11 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.

Attachment

General

Created:
Updated:
Size: