Page MenuHomePhabricator

Update DiscussionTools for warningbox changes
Closed, ResolvedPublic


The changes in T300306 might require some changes in DiscussionTools. We have some CSS (added in T289354) that hides some duplicate warning boxes in the new topic tool.

Event Timeline

Change 761462 had a related patch set uploaded (by Bartosz Dziewoński; author: Bartosz Dziewoński):

[mediawiki/extensions/DiscussionTools@master] Update references to warningbox class

Change 761462 merged by jenkins-bot:

[mediawiki/extensions/DiscussionTools@master] Update references to warningbox class

matmarex edited projects, added Editing QA; removed Patch-For-Review.

QA steps: Re-check T289354 and T289354#7312789 and confirm that the messages don't appear in two places when using the new topic tool.

For T289354, the deletion log is shown once as expected. See

Screenshot 2022-02-14 at 15.14.03.png (1×3 px, 288 KB)

As well as T289354#7312789. See

Screenshot 2022-02-14 at 15.15.21.png (1×1 px, 189 KB)