Page MenuHomePhabricator

Structured tasks: Post-edit message widget uses Success style only when an edit is made, else use the Notice style
Closed, ResolvedPublic

Description

This is a visual change made for the Post-edit dialogs in for Add images that should also apply to the Add links task.

Currently regardless of whether an edit is made when an "Add a link" or "Add a suggested image" structured task is submitted, we show the green Success messageWidget.

This task is to change the message style to the more neutral "Notice" style with Base80 background and Base10 icon when there is no edit made (if all links and image is rejected):

No edit made
image.png (1×720 px, 231 KB)
Edit published
image.png (1×720 px, 205 KB)

Event Timeline

Please add project tags as project tags, not as subscribers. Thanks.

This was done in T293316: Add an image: minimal quality gate. The changes are now in betalabs.

Add link

Screen Shot 2021-11-17 at 2.08.56 PM.png (764×756 px, 160 KB)

Add image

Screen Shot 2021-11-17 at 2.09.43 PM.png (778×756 px, 72 KB)

Hi @mewoph - sorry I missed this in my notifications but this task is about changing the message style to the Base80 background "Notice" style when there is *no* edit puslished, but it seems the opposite change has been made here where the green success style has been removed from both.

Thanks for the clarifications @RHo! I only checked the rejected state, so I didn't realize the same was happening to accepted state. For this particular task, I'll update the notice style. The copy will be fixed separately in T296929: [Regression] Incorrect "publish": message on Add a link mobile when there is not edit made .

Currently the issue where the notice style was shown instead of success only occurs when wgEditSubmitButtonLabelPublish config is true.

Change 743248 had a related patch set uploaded (by MewOphaswongse; author: MewOphaswongse):

[mediawiki/extensions/GrowthExperiments@master] Post-edit dialog: Show success MessageWidget whenever an edit is made

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

Change 743248 merged by jenkins-bot:

[mediawiki/extensions/GrowthExperiments@master] Post-edit dialog: Show success MessageWidget whenever an edit is made

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