Closed
Bug 949857
Opened 12 years ago
Closed 12 years ago
[Gallery]Red 'Delete' shown when deleting in multiple selection mode, blue "Ok" shown when deleting in fullscreen mode
Categories
(Firefox OS Graveyard :: Gaia::Gallery, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: ericcc, Assigned: johnhu)
References
Details
(Whiteboard: [fxos-bug-bash-1.2])
Attachments
(3 files)
Description:
Red 'Delete' shown when deleting in multiple selection mode, blue "Ok" shown when deleting in fullscreen mode
Steps to Reproduce:
1. Open a photo in Gallery, full screen mode, and delete it
2. Blue "Ok" shown for confirmation.
3. Enter multiple selection mode.
4. Select at least one photo, and delete it/them.
5. Red "Delete" shown for confirmation.
Expected Results:
* Same for both
Actual Results:
* Blue "Ok" shown for confirmation in full screen mode.
* Red "Delete" shown for confirmation in multiple selection mode.
Additional Notes:
Environmental Variables:
Version: 1.2
Device: Buri
Build Date: 12/12/2013
RIL Type (Mozilla RIL or Commericial RIL): Commericial RIL
Reporter | ||
Comment 1•12 years ago
|
||
Assignee | ||
Updated•12 years ago
|
Assignee: nobody → johu
Assignee | ||
Comment 2•12 years ago
|
||
Attachment #8347848 -
Flags: review?(dflanagan)
Comment 3•12 years ago
|
||
Comment on attachment 8347848 [details] [review]
Use the same confirm dialog
r- because you didn't remember to git add dialogs.js so this patch won't actually run.
If it was me, I think I'd keep this patch simpler and just call showConfirmDialog() from frames.js without refactoring to a separate file. I like the idea of having dialogs.js, but would like to see the overlay display code put into that file too, so it seems like a separate refactoring.
If you want to go ahead with the dialogs.js refactor here (and it is okay with me if you do), I'd suggest that you call the function Dialogs.confirm() instead of Dialogs.showConfirm().
Attachment #8347848 -
Flags: review?(dflanagan) → review-
Assignee | ||
Comment 4•12 years ago
|
||
Comment on attachment 8347848 [details] [review]
Use the same confirm dialog
Sorry about that. And the movement of overlay is a good idea. I had moved it to dialogs. Please review it again. Thanks.
Attachment #8347848 -
Flags: review- → review?(dflanagan)
Comment 5•12 years ago
|
||
Sorry to hold this up. I won't be able to review it until January.
Assignee | ||
Comment 6•12 years ago
|
||
Sure, no problem. This is not an urgent bug.
Comment 7•12 years ago
|
||
Comment on attachment 8347848 [details] [review]
Use the same confirm dialog
Looks good to me, though I did not test. One nit on github to optionally fix.
Attachment #8347848 -
Flags: review?(dflanagan) → review+
Assignee | ||
Comment 8•12 years ago
|
||
merged to master:
https://github.com/mozilla-b2g/gaia/commit/ec1064218fef0fadcead7a64a9ae423b655c71d0
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•