[wp-trac] [WordPress Trac] #27388: Make TinyMCE modal behavior more consistent

[WordPress Trac]

#27388: Make TinyMCE modal behavior more consistent
-------------------------+-----------------------------
 Reporter:  eablokker    |      Owner:
     Type:  enhancement  |     Status:  new
 Priority:  normal       |  Milestone:  Awaiting Review
Component:  TinyMCE      |    Version:  trunk
 Severity:  normal       |   Keywords:
  Focuses:  ui           |
-------------------------+-----------------------------
 TinyMCE modals currently have inconsistent behavior. The main modals I
 have identified are the add media (a special case), insert link, special
 character, and help.

 Behavior: Fade in on open
 Special character and help fade in on open
 Add media and insert link do not fade in

 My opinion: remove the fade in. It slows down the interaction and we don't
 see fade in anywhere else in Wordpress UI

 Behavior: Close on escape
 All modals close on escape, however special character and help do not
 close on escape if the modal loses focus, i.e. you click on the dark
 overlay background first

 My opinion: fixing the next issue will likely make this issue irrelevant

 Behavior: Close on click of dark background overlay
 Add media and insert link close on background click
 Special character and help do not close on background click

 My opinion: modal should always close when clicking background

 Bahavior: Modal is draggable
 Special character and help are draggable
 Add media and insert link are not draggable

 My opinion: Not sure but there should be some consistency. What benefit
 does draggability afford? I can see insert link benefitting from
 draggability because you might want to look at the link text in the editor
 window to remind you what you were linking to. I can see why insert link
 is not currently draggable, due to the centered positioning and the link
 to existing content flyout. Maybe when you click that, it can just re-
 center the modal in the page while retaining draggability.

--
Ticket URL: <https://core.trac.wordpress.org/ticket/27388>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform