Closed Bug 1077412 Opened 5 years ago Closed 5 years ago

Add a confirmation dialog when deleting contacts in the Desktop client

Categories

(Hello (Loop) :: Client, defect)

defect
Not set
Points:
1

Tracking

(firefox34 verified, firefox35 verified)

VERIFIED FIXED
mozilla35
Iteration:
35.3
Tracking Status
firefox34 --- verified
firefox35 --- verified

People

(Reporter: Paolo, Assigned: Paolo)

References

Details

(Whiteboard: [contacts, first release needed][loop-uplift])

Attachments

(1 file, 1 obsolete file)

Follow-up from bug 1038257 comment 22.
Flags: qe-verify+
Flags: firefox-backlog+
Attached patch The patch (obsolete) — Splinter Review
Attachment #8501048 - Flags: review?(mdeboer)
Paolo -- can we uplift this to Fx 34 or does it a string change?  I thought we couldn't uplift it because we didn't have the string landed prior to the Fx34 string freeze.
Flags: needinfo?(paolo.mozmail)
Comment on attachment 8501048 [details] [diff] [review]
The patch

Review of attachment 8501048 [details] [diff] [review]:
-----------------------------------------------------------------

r=me with comment addressed.

Thanks!

::: browser/components/loop/MozLoopAPI.jsm
@@ +298,5 @@
> +            null, null, {});
> +
> +          callback(null, chosenButton == 0);
> +        } catch (ex) {
> +          callback(ex);

this won't work, you need to do `cloneErrorObject(ex, targetWindow)`
Attachment #8501048 - Flags: review?(mdeboer) → review+
(In reply to Maire Reavy [:mreavy] (Plz needinfo me) from comment #2)
> Paolo -- can we uplift this to Fx 34 or does it a string change?  I thought
> we couldn't uplift it because we didn't have the string landed prior to the
> Fx34 string freeze.

The strings were already uplifted for this change. The patch doesn't contain new strings.
Flags: needinfo?(paolo.mozmail)
Attached patch Updated patchSplinter Review
Attachment #8501048 - Attachment is obsolete: true
Whiteboard: [contacts, first release needed][loop-uplift] → [contacts, first release needed][loop-uplift][checkin-needed to fx-team]
Blocks: 1075203
https://hg.mozilla.org/integration/fx-team/rev/fda842d4d372
Keywords: checkin-needed
Whiteboard: [contacts, first release needed][loop-uplift][checkin-needed to fx-team] → [contacts, first release needed][loop-uplift]
https://hg.mozilla.org/mozilla-central/rev/fda842d4d372
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla35
Verified fixed FF 35.0a1 (2014-10-08) Win 7, Ubuntu 13.04, OS X 10.9.5.
Perhaps it would have been better not to close the Loop panel when showing the confirmation dialog.
Status: RESOLVED → VERIFIED
Comment on attachment 8501154 [details] [diff] [review]
Updated patch

Approval Request Comment
Part of the staged Loop aurora second uplift set
Attachment #8501154 - Flags: review+
Attachment #8501154 - Flags: approval-mozilla-aurora?
Flags: needinfo?(paul.silaghi)
Verified fixed FF 34b1 Win 7
Flags: needinfo?(paul.silaghi)
Comment on attachment 8501154 [details] [diff] [review]
Updated patch

Already landed in aurora, approving for posterity
Attachment #8501154 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
You need to log in before you can comment on or make changes to this bug.