Opened 15 months ago

Closed 15 months ago

Last modified 15 months ago

#20096 closed enhancement (fixed)

Use native TinyMCE popup for the image properties dialog

Reported by: azaozz Owned by: azaozz
Priority: normal Milestone: 3.4
Component: TinyMCE Version:
Severity: normal Keywords:
Cc:

Description

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

Attachments (1)

close-mce-popups.patch​ (753 bytes) - added by azaozz 15 months ago.

Download all attachments as: .zip

Change History (6)

comment:1   azaozz — 15 months ago

  • Owner set to azaozz
  • Resolution set to fixed
  • Status changed from new to closed

In [19977]:

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

comment:2   mdawaffe — 15 months ago

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

comment:3   azaozz — 15 months ago

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 :)

comment:4 follow-up: ↓ 5   mdawaffe — 15 months ago

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

azaozz — 15 months ago

comment:5 in reply to: ↑ 4   azaozz — 15 months ago

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).

Version 0, edited 15 months ago by azaozz (next)
Note: See TracTickets for help on using tickets.