Change #1098067 had a related patch set uploaded (by Esanders; author: Esanders):

[oojs/ui@master] [BREAKING CHANGE] Make ButtonMenuSelectWidget a PopupButtonWidget

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

In T380860#10357273, @kostajh wrote:

This was done deliberately in rEREI404454b38e8c: form: Show dialog to visit Special:ChangeEmail but I also agree that there is no need to use the large size overrides here:

.ext-reportincident-emaildialog {
	font-size: @font-size-large;
	line-height: @line-height-medium;
}

@KColeman-WMF would it be OK to remove these overrides?

Change #1098017 had a related patch set uploaded (by Kosta Harlan; author: Kosta Harlan):

[mediawiki/extensions/ReportIncident@master] EmailAlertDialog: Remove font size and line height overrides

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

OOUI's ButtonMenuSelectWidget doesn't wrap the menu in a PopupWidget, which is where we previously implemented the auto-flip code.

You are unlikely to see it on mobile as the reply widget always appears on its own line

This is specific to the desktop implementation, right? I don't see the same issue on Minerva.

This was done deliberately in rEREI404454b38e8c: form: Show dialog to visit Special:ChangeEmail but I also agree that there is no need to use the large size overrides here:

Sounds good. Let's see if we can get to it in the next release.

Sounds good, I believe we've started documentation here: https://www.mediawiki.org/wiki/Extension:ReportIncident

OK, I think that answers the questions in the task description. @Madalina, might be worth updating a mediawiki.org page for this project to add to the requirements/description of what the extension does.

In T347280#10304892, @Madalina wrote:

So what will we be logging exactly? Just the username ? And are we talking about logging when they click the report button or just the actual submissions of reports?

Yes, reporting a comment from an IP editor / Temp Account should be allowed as they might also be emergency situations.

So what will we be logging exactly? Just the username ? And are we talking about logging when they click the report button or just the actual submissions of reports?

In T344641#10297013, @KColeman-WMF wrote:

Yes I agree this task is no longer important. We have reworked the designs to include footer text making it explicit to users when they are filing a report. This is something we can include after pilot wiki launch.

In T346680#10296315, @kostajh wrote:

@KColeman-WMF @Madalina please review this and let us know if anything in the existing implementation needs to change.

Yes I agree this task is no longer important. We have reworked the designs to include footer text making it explicit to users when they are filing a report. This is something we can include after pilot wiki launch.

I assume this is done (please add any relevant links if you'd like)

I assume this is done (but no description?) so I am resolving it. If there are links to share with this task @Madalina, please add them.

I assume this is done, so I am resolving it.

@KColeman-WMF @Madalina please review this and let us know if anything in the existing implementation needs to change.

@KColeman-WMF @Madalina is this task still important? My 2c, I would not include this for pilot wiki launch and think just letting "X" close the dialog without any other prompts is sufficient.

We did something like this in T338817: [M] Limit the number of characters a user can add when reporting

cc @KColeman-WMF @Madalina