Remember position of Python editor windows & fix crash when deleting a Python node with an open editor

[Dewb]

Purpose

Python editor windows that are moved or resized from the initial location will remember their new size/position for the duration of the Dynamo session. This size information will not be persisted to the workspace, so closing and reopening the workspace, or Dynamo, will reset the window locations back to their default position.

If an editor window is on a secondary display when it is closed, and the secondary display is disconnected, the editor window will be restored to the default position if it is opened.

This also fixes a crash issue when the user deletes a Python node while the editor window is still open. The user will be shown a warning dialog giving them the opportunity to cancel the deletion, or hit OK to close the window and delete the node. (QNTM-2533).

Declarations

Check these if you believe they are true

• The code base is in a better state after this PR
• Is documented according to the standards
• The level of testing this PR includes is appropriate
• User facing strings, if any, are extracted into *.resx files
• All tests pass using the self-service CI.
• Snapshot of UI changes, if any.
• Changes to the API follow Semantic Versioning, and are documented in the API Changes document.

Reviewers

@aparajit-pratap

FYIs

@Racel

[Racel]

Thanks for doing this @Dewb. Just linked it to the OP about resizing Python windows. When this is merged, we should close that issue.

Also, would you mind adding images of the UI changes to the description? We typically do this for all UI changes per the declaration checklist.

[aparajit-pratap]

@Dewb LGTM!

[Dewb]

@Racel image added!

[mjkkirschner]

Hey @Dewb I'm just curious what happens when you close a python node on an external monitor, and then later reopen it when the monitor doesn't exist?

[Dewb]

@mjkkirschner I finally got some "free time" (ha!) today to investigate your very good question, and as I expected, the window is restored offscreen in that situation. Just pushed a fix. It's been tested on my 3-display desktop, but it could use some additional testing on a high-DPI display. (Updated the description with this feature.)

[mjkkirschner]

@Dewb I will try to test this this weekend.

[Racel]

@mjkkirschner - is this good to merge?

[mjkkirschner]

@Racel LGTM