Add a clearer warning message before deleting a site from a network

[johnbillion]

Deleting a site from the Network Admin -> Sites screen is the most destructive single action that a user can take on a WordPress installation.

After clicking the Delete link the user is shown a screen requesting confirmation, but this screen doesn't convey the permanence of the action they're about to take.

There is no trash or undo when deleting a site. It drops the database tables for the site, empties its uploads directory of all files, and removes the roles for that site from all users.

Let's add an extra warning to this confirmation screen which explains this.

[johnbillion]

51358.diff​ adds an extra warning to the confirmation screen when deleting a site, and changes the submit button text for further clarity.

[johnjamesjacoby]

This is a big +1 from me. I love the patch. It’s a nice improvement.

I’d like to see more here! What if we added a little summary of all the things you mentioned in the ticket description? An indication of the irreversible destruction that’s about to happen might provide enough context to make this screen perform its intended function better, avoiding unintended catastrophic data obliteration.

Deleting this site will irreversibly delete the following:

• 200 posts
• 64Mb of media
• 7 user associations

The following plugins are currently active on this site. Use these to help with deciding if deleting this site is still OK.

• bbPress
• Sugar Calendar

[kebbet]

There is a strive towards using sentence case in core from my understanding. I think the button text should be “Delete this site permanently”.

Thanks!

[johnbillion]

@kebbet Thanks, is that documented somewhere? I copied the button text from the button on the Tools -> Delete Site screen.

@johnjamesjacoby That's a really interesting idea but I wonder whether it would make the screen a bit too busy?

[kebbet]

@johnbillion I guessed that question was coming ;) A quick search on trac for ”sentence case” results in some tickets, but I could not find an announcement on the topic.
whyisjake states it here e.g. https://core.trac.wordpress.org/ticket/40244#comment:22
Another ticket on the topic #51371

[kebbet]

My first patch ever, downloaded the previous one and edited it. Maybe I did it in an incorrect way, happy for any feedback :D

[johnbillion]

I just realised this doesn't cover bulk site deletion.

[hellofromTonya]

During today's scrub, @johnbillion noted there's more work to do and to punt this one to Future Release.

If any maintainer or committer feels this can be resolved in time, or wishes to assume ownership during a specific cycle, feel free to update the milestone accordingly.

[cafenoirdesign]

Refresh for existing patch

[Mista-Flo]

Hey @cafenoirdesign, good patch, thanks for the work, it looks pretty solid.

That being said, I'm not sure about the use of <b> tag, it became pretty standard to use <strong> instead if you wanna keep it.

[cafenoirdesign]

A patch with strong tags instead of <b> tags.

[audrasjb]

As per today's bug scrub and since tomorrow is WP 6.5 beta 1, let's move this ticket to Future Release to give it time for testing.

[johnbillion]

In 58098:

Networks and Sites: Add a clearer warning message before deleting a site from a network.

Props johnjamesjacoby, kebbet, cafenoirdesign, Mista-Flo, johnbillion

Fixes #51358