Closed Bug 1275383 Opened 4 years ago Closed 4 years ago

Improve Sync disconnection warning

Categories

(Firefox :: Sync, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
Firefox 49
Tracking Status
firefox49 --- fixed

People

(Reporter: rfeeley, Assigned: eoger)

Details

Attachments

(2 files)

Am hoping to have your feedback on this dialog that appears when disconnecting Sync. Our copy audit pointed out this dialog specifically as problematic. How is this?

CURRENT:
Are you sure?
Firefox will stop syncing with our account, but won’t delete any of your browsing data on this computer.
- Continue (primary)
- Cancel

PROPOSED:
Disconnecting from Sync means that this computer will no longer sync with your account, however your browsing data will still remain on this computer.
- Disconnect (primary)
- Cancel
I think we could simplify it a bit:

Disconnect from Sync? 
Your browsing data will remain on this computer, but it will no longer sync with your account.
- Disconnect (primary)
- Cancel

How does that sound?
Better, but is "it" a little ambiguous?
Flags: needinfo?(matej)
(In reply to Ryan Feeley [:rfeeley] from comment #2)
> Better, but is "it" a little ambiguous?

I don't think so. Grammatically it refers to "your data," but even if some reads it as referring to "this computer," that's still accurate.
Flags: needinfo?(matej)
True! Discussion is now over (unless Mark or Eduoard have something to add). I have cc:ed them to implement when possible.
Assignee: matej → nobody
Summary: Discuss improving Sync disconnection warning → Improve Sync disconnection warning
Flags: firefox-backlog+
Assignee: nobody → edouard.oger
Status: NEW → ASSIGNED
Comment on attachment 8758936 [details]
MozReview Request: Bug 1275383 - Change the copy of Sync disconnection warning. r?markh

https://reviewboard.mozilla.org/r/57016/#review53796

Great, thanks!

::: browser/locales/en-US/chrome/browser/syncSetup.properties:55
(Diff revision 1)
>  # Several other strings are used (via Weave.Status.login), but they come from
>  #  /services/sync
>  
>  # Firefox Accounts based setup.
>  continue.label = Continue
> +disconnect.label = Disconnect

Let's add a couple of "# LOCALIZATION NOTE" comments here - note that this is the label for the disconnect button.

::: browser/locales/en-US/chrome/browser/syncSetup.properties:56
(Diff revision 1)
>  #  /services/sync
>  
>  # Firefox Accounts based setup.
>  continue.label = Continue
> +disconnect.label = Disconnect
>  disconnect.verify.title = Disconnect

and these these are used in the confirmation dialog shown when the user hits the disconnect button.
Attachment #8758936 - Flags: review?(markh) → review+
Comment on attachment 8758936 [details]
MozReview Request: Bug 1275383 - Change the copy of Sync disconnection warning. r?markh

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/57016/diff/1-2/
Keywords: checkin-needed
Pushed by cbook@mozilla.com:
https://hg.mozilla.org/integration/fx-team/rev/bf9266fa9187
Change the copy of Sync disconnection warning. r=markh
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/bf9266fa9187
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 49
Reproduced it on Firefox nightly according to(2016-05-24)

Fixing bug is verified on Latest Developer Edition--Build ID:( 20160703004019 ), User Agent: Mozilla/5.0 (Windows NT 10.0; rv:49.0) Gecko/20100101 Firefox/49.0

After changing Sync disconnection warning It seems oke on new version.

Tested OS-- windows10 32bit
QA Whiteboard: [bugday-20160629]
You need to log in before you can comment on or make changes to this bug.