Yes, but only the diff.

What if my change is one parameter in a function call ? The most meaningful aspect of the change lies in the definition of said function, that may not be part of the diff.

What if my change is related to another commit that I want to reference in my message ? I’d have to close the dialog, copy the commit ID, and reopen the dialog.

For an infinite amount of reasons we rarely foresee, blocking states are never good.

If I wanted VSCode, I’d use VSCode and their shitty design.

Same.

But this one thing isn’t part of said shitty design.

Honestly, that was a good move. JetBrains used to have many actions as popups that prevent reviewing the source code and the project in general while e.g. figuring out a commit message.