Change 684312 had a related patch set uploaded (by STran; author: STran):

[mediawiki/core@master] [WIP] Update Special:Block UI

https://gerrit.wikimedia.org/r/684312

Sitewide blocks include default actions like moving pages, sending thanks, uploading files etc.

We should remove "sending thanks" from this help text, since it only makes sense if the Thanks extension is enabled.

In T280719#7057434, @Tchanders wrote:

Sitewide blocks include default actions like moving pages, sending thanks, uploading files etc.

We should remove "sending thanks" from this help text, since it only makes sense if the Thanks extension is enabled.

Agree.

Test wiki created on Patch Demo by TChan (WMF) using patch(es) linked to this task:

https://patchdemo.wmflabs.org/wikis/8eacf7d3d3/w/

In T280719#7057743, @PatchDemoBot wrote:

Test wiki created on Patch Demo by TChan (WMF) using patch(es) linked to this task:

https://patchdemo.wmflabs.org/wikis/8eacf7d3d3/w/

@Prtksxna @Niharika We realised that it's impractical to do this redesign behind a feature flag, since so many bits and pieces need updating.

I've created a patch demo so that you can have a quick look before we move forward. Available user accounts:

You can log in as the following users using the password patchdemo1
Patch Demo (admin)
Alice
Bob
Mallory (blocked)

Something I noticed testing a different patch on patchdemo (without this redesign, but with action blocks enabled):

Load Special:Block
Uncheck "editing", and select a couple of action blocks
Change the expiration dropdown selection

Expected result: nothing happens to the rest of the form
Actual result: action blocks are all unselected

In T280719#7059228, @DannyS712 wrote:

Something I noticed testing a different patch on patchdemo (without this redesign, but with action blocks enabled):

Load Special:Block
Uncheck "editing", and select a couple of action blocks
Change the expiration dropdown selection

Expected result: nothing happens to the rest of the form
Actual result: action blocks are all unselected

Thanks - that's the known bug from: T279556#7041719

We've discussed and decided it's not worth fixing since the design is about to change, and it doesn't interfere with QA.

From gerrit. reformatted a bit, but its a bit messy

I left a bunch of notes, but in general I don't think this is an improvement to the UI either with or without action blocks being enabled. I created 3 patchdemos for testing (the relevant fourth, no action blocks and without this patch, is just production) [1][2][3] and tried them out - based on my own experience with blocks as a sysop, the old interface was more intuitive. But, that might just be me - can I suggest soliciting feedback from other sysops?

--------
Testing without action blocks being enabled, its confusing that the options for "Account creation", "Sending email", and "Editing their own talk page" jump around to be after whichever option of sitewide/partial is chosen. This becomes even more confusing when, for action blocks being enabled, they jump to the middle of the partial blocks display.

-------
The prior layout (with action blocks enabled) was

Actions to block:
[] Editing
  o Sitewide
    Every page on the wiki and all other actions

  o Partial
    Specific pages or namespaces
    Pages
    [_____]
    Namespaces
    [_____]

[] Account creation
[] Sending email
[] Editing their own talk page
[] Uploading files
[] Renaming pages
[] Creating new pages
and its now

Actions to block:
* Sitewide
 Sitewide blocks include default actions like moving pages, uploading files, etc.
   [] Account creation
   [] Sending email
   [] Editing their own talk page

o Partial
or

Actions to block:
o Sitewide
 Sitewide blocks include default actions like moving pages, uploading files, etc.

* Partial
   Pages
   [_____]
   Namespaces
   [_____]
   [] Account creation
   [] Sending email
   [] Editing their own talk page
   [] Uploading files
   [] Renaming pages
   [] Creating new pages
but "Sitewide" and "Partial" are not "Actions to block", and should not be listed as direct radio options under that heading. Rather, the "Actions to block" refers to
- whether to block editing and actions that include editing, either sitewide or limited to specific pages and namespaces, as well as specific actions that include editing (uploading/moving/creating pages)
- account creation
- sending email
- editing own talk page
and it was much easier to understand that with the old layout

Also:

- The new message for sitewide blocks is less helpful, and the removal of the help message for partial blocks is undesirable - it should just be updated to include that actions can be blocked
- sitewide blocks also include editing - it was clearer with the old layout of a dedicated section to editing sitewide vs partially, especially since this was removed from the help message
- what are "default actions"? A partial block, by default, does not apply to any actions, but sitewide blocks apply by default to *all actions*  that are included in partial block actions (but not actions like sending email, unless selected separately)

----
I'll also note that hiding the partial blocks UI when sitewide blocks are selected was previously proposed and shot down, see [4][5], and (as both the developer who sent the prior patch to hide things, and a sysop on multiple WMF wikis where I block frequently) I'm slightly in favor of not hiding things now, it makes the interface easier to use when all of the partial block options are visible when you're making the decision between sitewide and partial, especially as action blocks mean that partial blocks can be used more and more. Otherwise, you might have a sysop that wants to sitewide block because someone is renaming pages disruptively, but doesn't see that that is an option in the partial block interface because its not shown at first, and so they just sitewide block

[1] With action blocks, without this redesign: https://patchdemo.wmflabs.org/wikis/b5a1fd1da7/wiki/Main_Page
[2] With action blocks, with redesign: https://patchdemo.wmflabs.org/wikis/97f509e95e/wiki/Main_Page
[3] No action blocks, with redesign: https://patchdemo.wmflabs.org/wikis/a3a2f92793/wiki/Main_Page
[4] https://gerrit.wikimedia.org/r/c/mediawiki/core/+/537718
[5] https://phabricator.wikimedia.org/T217363

Change 685908 had a related patch set uploaded (by STran; author: STran):

[mediawiki/core@master] Stop resetting partial action block options when values change

https://gerrit.wikimedia.org/r/685908

Editing checkbox

sitewide blocks also include editing - it was clearer with the old layout of a dedicated section to editing sitewide vs partially, especially since this was removed from the help message

I think we should remove the "Editing" checkbox, since it can mislead admins into thinking that partial blocks don't block any actions.

• Sitewide blocks block editing and a load of other actions
• Partial blocks block editing and a some other actions relating to the blocked pages/namespaces (e.g. move, thanks)

A partial block, by default, does not apply to any actions, but sitewide blocks apply by default to *all actions* that are included in partial block actions (but not actions like sending email, unless selected separately)

Partial blocks do block actions, but it's totally understandable that admins may not realise this since the form currently implies that they don't.

The editing checkbox also adds unnecessary clutter to the form, and also extra unnecessary logic to the code.

Hiding/showing fields

I think we had an iteration of the action block design where we managed to avoid this.

It involved putting the create/upload/etc checkboxes under the "partial" radio along with the "pages" and "namespaces" widgets.

• Cons: it splits up the actions
• Pros: disable/enable better for a11y; easier to use in PHP-only (as @STran found, the current design is difficult for PHP-only users); makes it clear that two sets of actions are treated differently

It looked / could look something like this:

Block type

* Sitewide
      Block every page on the wiki and all contribution actions.

* Partial
   Pages
       Block editing and some contribution actions on certain pages
   [_____]
   Namespaces
       Block editing and some contribution actions on certain namespaces
   [_____]
   Actions
       Block certain actions on all pages
   [] Uploading files
   [] Renaming pages
   [] Creating new pages

Also block these actions

[] Account creation
[] Sending email
[] Editing their own talk page

@DannyS712 Thank you for your thoughtful feedback. @Prtksxna, @Tchanders and I discussed these changes at length yesterday and agree with the points you made. We'll have some new mocks to share with the community (on this page) and will seek feedback on it before making any widespread changes.

For awareness, our process behind this change was (is) to build the changes behind a feature flag and enable them on beta/testwiki environments where we can invite community members to see the changes and give feedback. However in this case it turned out that it was not really an option to contain the redesign behind a feature flag and so the feedback process has become more complicated. Rest assured, we would not push anything through without consulting the community.

FYI on this task - we have an active discussion on the meta page about the proposed designs.

@STran @TThoabala This is a good task for us to finish up now that SecurePoll is sorta past us. There has been a request from a Czech wiki user that they would like to pilot action blocks on their wiki.

This help page will need to be created and should explain what kinds of actions are blocked on sitewide.

@Niharika @Prtksxna I don't suppose we made this already? 😅 If not, what would next steps to getting this to exist be? Thanks!

I don't think there's any harm in keeping this as part of this ticket unless it becomes a blocker. Would I be the one writing the copy for it?

In T280719#7332399, @STran wrote:

I don't think there's any harm in keeping this as part of this ticket unless it becomes a blocker. Would I be the one writing the copy for it?

If you can. I can help but I am not sure I can authoritatively list out all the actions that will be blocked with a sitewide-block.

Change 684312 merged by jenkins-bot:

[mediawiki/core@master] Update Special:Block UI

https://gerrit.wikimedia.org/r/684312

@STran: I left two tiny nitpicks on https://gerrit.wikimedia.org/r/684312 but only after the patch had been merged. Would you mind taking a quick look them?

Change 722655 had a related patch set uploaded (by STran; author: STran):

[mediawiki/core@master] Update block option heading text to match design

https://gerrit.wikimedia.org/r/722655

Change 722655 merged by jenkins-bot:

[mediawiki/core@master] Update block option heading text to match design

https://gerrit.wikimedia.org/r/722655

The behaviour is consistent with all the acceptance criteria in the description.

• Show sub-text (always visible): Blocks editing on all pages and namespaces. Also blocks default actions (See help)

...

• Show sub-text (always visible): Allows you to select pages and namespaces to block. Doesn’t block default actions but lets you block certain specific actions.

I cannot find anywhere where we define what the "default" actions are.

I tested submitting blocks with different parameters on beta and testwiki. I also edited pre-existing blocks, in case there are backward-compatibility problems.

I checked that the block parameters were correctly recorded in the database on beta. I also checked that there were no exceptions on beta and testwiki.

I also tested on beta with JavaScript disabled.

I also briefly tested with $wgEnablePartialActionBlocks = false on my local docker, so the new action blocks don't appear.

Test environments:

• https://test.wikipedia.org MediaWiki 1.38.0-wmf.1 (rMW667d96c1b1a1) 19:25, 23 September 2021.
• https://en.wikipedia.beta.wmflabs.org MediaWiki 1.38.0-alpha (edad7b3) 09:15, 24 September 2021.
• Local docker MediaWiki 1.38.0-alpha (edad7b3) 09:15, 24 September 2021.

I cannot find anywhere where we define what the "default" actions are.

@Prtksxna do you recall what the default actions are?

In T280719#7402815, @Niharika wrote:

I cannot find anywhere where we define what the "default" actions are.

@Prtksxna do you recall what the default actions are?

I don't think we ever had a complete list. IIRC, @Tchanders told us that there are a bunch of if-conditions that get called in MediaWiki and extensions that defines this, not a separate flag etc. In this ticket we've only used examples like - moving pages, sending thanks, uploading files etc

Test wiki on Patch demo by TChan (WMF) using patch(es) linked to this task was deleted:

https://patchdemo.wmflabs.org/wikis/8eacf7d3d3/w/