WordPress.org

Make WordPress Core

Opened 7 weeks ago

Closed 7 weeks ago

#48900 closed enhancement (fixed)

Rename Comment Blacklist option to be more inclusive

Reported by: dartiss Owned by: jorbin
Milestone: 5.4 Priority: normal
Severity: normal Version:
Component: Text Changes Keywords:
Focuses: Cc:
PR Number:

Description

At the moment, certainly in the English translation, there is a "Comment Blacklist" field (Settings -> Moderation in WP Admin).

To make this more inclusive, I would like to see this renamed. My suggestion would be "Blocklist" but I'm sure there are others that would equally work.

Attachments (1)

48900.diff (1008 bytes) - added by jorbin 7 weeks ago.

Download all attachments as: .zip

Change History (6)

@jorbin
7 weeks ago

#1 follow-up: @jorbin
7 weeks ago

  • Milestone changed from Awaiting Review to 5.4

I agree the UI should absolutely be updated here. I don't want to change the underlying option as that is a backcompat breakage (unless we added code to handle that, but we would still be in the situation where blacklist is supported in code).

#2 in reply to: ↑ 1 @dartiss
7 weeks ago

Replying to jorbin:

I agree the UI should absolutely be updated here. I don't want to change the underlying option as that is a backcompat breakage (unless we added code to handle that, but we would still be in the situation where blacklist is supported in code).

Totally agree.

#3 @dartiss
7 weeks ago

Just noting here too, that once this change is live, we will need to update the documentation that the settings page links to - https://wordpress.org/support/article/settings-discussion-screen/#comment-blacklist.

#4 @jeremyfelt
7 weeks ago

48900.diff catches the screenreader text, but does not get the text displayed on the discussion settings page—one line above.

#5 @jorbin
7 weeks ago

  • Owner set to jorbin
  • Resolution set to fixed
  • Status changed from new to closed

In 46820:

Text Changes: Rename Comment Blacklist option to be more inclusive

This doesn't change the underlying option name as that would be a backcompat break, but it makes the UI a bit more inclusive.

Props dartiss, jeremyfelt, aaronjorbin.
Fixes #48900.

Note: See TracTickets for help on using tickets.