Page MenuHomePhabricator

Convert flag reason form in QuizGame/PictureGame too OOjsUI dialog.
Closed, ResolvedPublic

Description

Hmm, if we are doing T155451 , why do not do it in OOjsUi? We would also use it in QuizGame.

Event Timeline

Go for it, I like that idea :-) A simple prompt dialog should do the trick.

Change 334378 had a related patch set uploaded (by Filip):
Converted old reason field, to OOjsUI dialog.

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

Change 334378 merged by jenkins-bot:
Converted old reason field to OOjsUI dialog.

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

First patch got merged - will upload same patch for quizgame tomorrow

Awesome to hear! Thank you for all you work :D Also, you mean PictureGame, right?

@Filip Forgot about PollNY, this will need a OOUI dialog as well for consistency. I can do this one if you don't want to or if you're busy :-)

@SamanthaNguyen: Ok, need to port PollNY - Will do it tomorrow, just need to add OOJSUI dialog

Change 340168 had a related patch set uploaded (by Filip; owner: Filip):
Converted alerts, and confirms to OOJSUI

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

Change 340168 merged by jenkins-bot:
[mediawiki/extensions/PollNY@master] Converted alerts, and confirms to OOJSUI

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

ashley removed a project: Patch-For-Review.