Cancel your account as a collapsible field

Code wise seems okay.

Hi there
@BramDriesen it would be nice if you attach a screenshot with a fieldset, And the result of deleting the user account using this fildset. Thanks a lot!

There is already an image attached :)

But you put RTBC, so you have to show that it works fine. That attached screenshot is from @xaiwant.

Since when is it mandatory to attach an image or show that something is working? Meaning someone put it on RTBC means that he/she tested and reviewed the code and marked it as ok. (Doesn't mean that the maintainer has to accept it though since there are some cases that you might want to reject it, e.g. wrong approach, out of scope etc).

I already destroyed the environment to test, and I'm not planning to boot a new one just for a screenshot...

If you're that willing to see a screenshot, put it back to needs review.

@BramDriesen, ok I understood your point of view. Thanks for rewriting.

Thank you @xaiwant for improvement. But I noticed some issues in your patch:

• Please use short array syntax to be in case with Drupal 8 best practices;
• Please use translate method from class ($this->t()) instead of t()

Could you fix them?

Hi @andrew_tspkh,

As instructed, I have made the changes in patch for improvement:

• Please use short array syntax to be in case with Drupal 8 best practices; Done
• Please use translate method from class ($this->t()) instead of t() : Done

cancel_aacount-3016062-11.patch please review.

Tested again and works fine. (I indeed missed the two coding standard fixes you noticed)

With screenshot this time ;)

After clicking cancel the account is correctly canceled.

Committed to 8.x-1.x branch.
Thanks to all for contributing.