The user cancellation texts are quite scary that all content is about to be removed if I cancel one user. Its pretty scary. We should instead say its the account's content. (Note that this is displayed on the user settings pages as general settings options, then when canceling an account or when canceling your own account). So we need text that fits all three scenarios.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

David_Rothstein’s picture

Maybe just "... all content belonging to it" for most of these?

RobLoach’s picture

Status: Needs review » Reviewed & tested by the community

In summary, we're changing the following text when a user account is being deleted:

  • Disable the account and keep all content. ---> Disable the account and keep the account's content.
  • Disable the account and unpublish all content. ---> Disable the account and unpublish the account's content.
  • Delete the account and make all content belong to the %anonymous-name user. ---> Delete the account and make the account's content belong to the %anonymous-name user.
  • Delete the account and all content. ---> Delete the account and the account's content.

I think it's a reasonable change and gives a bit more context to the text.

Maybe just "... all content belonging to it" for most of these?

".. all content belonging to it" might work too, but it would be nice if we could use her/him to give some respect as it kind of sounds weird in some cases.

  • Disable the account and keep the content belonging to it.
  • Disable the account and unpublish the content belonging to it.
  • Delete the account and make the %anonymous-name user own all content belonging to it. <----------- Weird
  • Delete the account and all content belonging to it.
David_Rothstein’s picture

Status: Reviewed & tested by the community » Needs review

Crosspost. We shouldn't repeat the word "account" - @ksenzee agrees.

RobLoach’s picture

Issue tags: +Needs usability review

Usability team to the rescue?

cbrookins’s picture

I don't think repeating the word account is a problem - it makes it crystal clear. However could replace 'the account' with 'it's'

* Disable the account and keep all content. ---> Disable the account and it's content.
* Disable the account and unpublish all content. ---> Disable the account and unpublish it's content.
* Delete the account and make all content belong to the %anonymous-name user. ---> Delete the account and make it's content belong to the %anonymous-name user.
* Delete the account and all content. ---> Delete the account and it's content.

David_Rothstein’s picture

Issue tags: +String freeze

Adding the "string freeze" tag.

joachim’s picture

#5 looks good, but obviously should be 'its' and not 'it's' ;)

sun’s picture

Yeah, let's just squeeze an "its" into all options. Sufficiently resolves this issue.

joachim’s picture

Here's a patch with 'its'. Like Monty Python ;)

David_Rothstein’s picture

Status: Needs review » Reviewed & tested by the community

Patch looks good - looks like a no-brainer to me.

I think we can maybe still squeeze this in as a bugfix. If not, marking it RTBC is the way for someone to tell us that :)

webchick’s picture

Status: Reviewed & tested by the community » Fixed

I think this is sufficiently unclear that we can classify as a bugfix.

Committed to HEAD. Thanks!

Status: Fixed » Closed (fixed)
Issue tags: -Needs usability review, -String freeze

Automatically closed -- issue fixed for 2 weeks with no activity.