Use native TinyMCE popup for the image properties dialog

[azaozz]

As the title, the native TinyMCE dialogs are better managed than thickbox and have extra features like being movable, resizable, etc.

[azaozz]

In [19977]:

Use native TinyMCE popup for the image properties dialog, fixes #20096, see #20094

[mdawaffe]

With thickbox dialogs, you can click outside of them and they go away. That is not the case with the TinyMCE dialog in [19977].

[azaozz]

Right but this one works better when MCE is on the front-end. We may be able to hack it to close when clicking the background too :)

[mdawaffe]

Not saying it's better or worse. Just pointing out a UI inconsistency.

[azaozz]

Replying to mdawaffe:

...Just pointing out a UI inconsistency.

Right, the "close on clicking the background" is a Thickbox thing, MCE's popups never had that. Seems easy enough to add it though, patch attached (may need a bit more testing).