[wp-trac] Re: [WordPress Trac] #7066: Press This major enhancements, ajax, tabs, photo loading

WordPress Trac wp-trac at lists.automattic.com
Wed Jun 4 21:34:35 GMT 2008


#7066: Press This major enhancements, ajax, tabs, photo loading
----------------------------+-----------------------------------------------
 Reporter:  noel            |        Owner:  ryan
     Type:  enhancement     |       Status:  new 
 Priority:  high            |    Milestone:  2.6 
Component:  Administration  |      Version:  2.6 
 Severity:  normal          |   Resolution:      
 Keywords:                  |  
----------------------------+-----------------------------------------------
Comment (by azaozz):

 I think there were two reasons not to use ui.tabs here: it doesn't play
 well with TinyMCE and the tag suggest, and it's slower compared to only
 loading the differences between the tabs and leaving the editor, the tags
 field and the categories in place as in the current version (also
 discussed in #6998).

 To make the press-this extensible, we could add a hook in the tabs that
 would reload the whole page inside the popup and display content supplied
 by a plugin, similarly to how the admin submenu/plugins settings pages
 work. That way all scripts would be unloaded correctly and initiated again
 when needed.

 Currently the add image functionality would need some refining. In most
 browsers it is possible to drag the small preview images to the editor and
 they will be inserted but with the wrong settings. I think many users
 would try doing that. There are ondrag, ondrop, etc. js events but the
 support for them varies a lot between the browsers. Perhaps we could try
 disabling this. Another option is to make a TinyMCE plugin that would
 handle it properly in the browsers that support it, and let it insert the
 html tags.

 Also clicking on a thumbnail in the images field shows second preview in
 Thickbox and at the same time inserts the image into the editor. Perhaps
 it would be more user friendly to either have "OK" and "Cancel" on the
 Thickbox preview or not show it at all, as it's a bit confusing.

 Don't think the "extended_valid_elements" setting is needed in TinyMCE.
 Most of it repeats the default rules and adds some non XHTML attributes.
 For handling video we could try including the "Media" plugin (as in the
 default editor settings). It has support for almost all current video
 formats and we don't need to include the button on the toolbar for it to
 work. It would produce the object/embed tags when given the url to a video
 file.

-- 
Ticket URL: <http://trac.wordpress.org/ticket/7066#comment:10>
WordPress Trac <http://trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list