[wp-trac] [WordPress Trac] #26952: Style TinyMCE modals to match WordPress admin styling

[WordPress Trac]

#26952: Style TinyMCE modals to match WordPress admin styling
--------------------------+--------------------
 Reporter:  melchoyce     |       Owner:
     Type:  defect (bug)  |      Status:  new
 Priority:  normal        |   Milestone:  3.9
Component:  TinyMCE       |     Version:  trunk
 Severity:  normal        |  Resolution:
 Keywords:                |     Focuses:  ui
--------------------------+--------------------

Comment (by avryl):

 > Still not sure about scrolling the whole "body" hiding the top part and
 then auto-scrolling to the top on clicking a search result in the bottom
 part. The old behavior is a better UX imho. Clicking on a search result
 fills the fields at the top, logically they should always be visible.

 I thought it would be better to allow more space for the list depending on
 your screen size. But yes, it'd be better if the input fields are visible
 after selecting a post. Scrolling to the top would be annoying, so it
 looks like the old scroll behaviour is better.

 > Also we lost the "Cancel" link […]

 I think we should get rid of it. No other modal has it.

 > Not so sure about the changed "looks" too. Patched it looks like the
 left side of the above image, was thinking it should be more like the
 right one. We would change the structural CSS, the header, and tweak the
 colors in the modal's "body". Not sure it needs anything else.
 > The "header" and "footer" feel very large too, especially when the modal
 is folded. I realize we are trying to follow the media modal styling but
 it covers nearly the whole screen and the footer there has another purpose
 (shows thumbnails). In all the rest of the modals the footer only holds
 the "submit" button.

 I agree that the headers look a bit big (was trying to copy it from the
 media modal). So let’s resize them to about 40px? I’d keep the colour of
 the header though, and remove the visual distinction of the footer.

--
Ticket URL: <https://core.trac.wordpress.org/ticket/26952#comment:49>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform