WordPress.org
Get WordPress

Make WordPress Core

Opened 15 years ago

Closed 15 years ago

Last modified 15 years 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: Focuses:

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 years ago.

Download all attachments as: .zip

Change History (6)

#1 @azaozz
15 years ago

  • Owner set to azaozz
  • Resolution → fixed
  • Status new → closed

In [19977]:

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

#2 @mdawaffe
15 years 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].

#3 @azaozz
15 years 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 :)

#4 follow-up: @mdawaffe
15 years ago

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

@azaozz
15 years ago

#5 in reply to: ↑ 4 @azaozz
15 years 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 to maintain consistency, patch attached (may need a bit more testing).

Last edited 15 years ago by azaozz (previous) (diff)
Note: See TracTickets for help on using tickets.