A new comment flagging modal just went live:
Aside from the fact that now it doesn't match the other flagging/closure modals in design (something that will likely be changed in the future...), I've already noticed a few things that, while minor, I don't like about it that I think could be improved.
It suffers from the new-age "animated is good" problem. Specifically, the 100ms transition feels slow and jarring and I would like to see it removed (yes, I know I can do this with a user script, but it would improve the responsiveness of the site and reduce their code base size by getting rid of this effect)
When inspecting it, I see the z-index value is set to 9000. This should be changed to 2 or something more semantic (Stack Exchange sites don't have elements on 9000 different z-indices, last I checked). In fact, changing it to 2 on my own didn't make it disappear behind other stuff. Some other modals may interfere, though, so 5 or 10 would be a safer bet than 2.
The middle flag option is now named "It's not relevant". I'm sure someone complained about this phrasing in the past when new comment reasons have been demo'd, but lots of comments that are relevant should still be removed. There was nothing wrong with the previous name of "No longer needed"... can we please have that phrasing back, or at least keep the spirit of it with "Not needed"? As it stands, "It's not relevant" is factually incorrect in some cases and repeated in the longer subheading.