[wp-trac] Re: [WordPress Trac] #3171: Don't hide advanced button

WordPress Trac wp-trac at lists.automattic.com
Mon Apr 23 21:07:19 GMT 2007


#3171: Don't hide advanced button
-------------------------+--------------------------------------------------
 Reporter:  mypatricks   |        Owner:  ryan    
     Type:  enhancement  |       Status:  reopened
 Priority:  normal       |    Milestone:          
Component:  TinyMCE      |      Version:  2.2     
 Severity:  normal       |   Resolution:          
 Keywords:               |  
-------------------------+--------------------------------------------------
Changes (by Otto42):

  * status:  closed => reopened
  * resolution:  invalid =>

Comment:

 +1

 Adding the advanced button to the bar gives some very handy options in
 2.1.3, including the "paste from text" and "paste from Word" buttons that
 would eliminate a whole lot of problematic usage. It also adds semantic
 pulldowns that people can use to make headings, paragraphs, etc. Not
 displaying that secondary bar by default is a good thing, admittedly, but
 relying on a keypress that's unusual (alt-shift-v in firefox) and not
 cross-browser (it's just alt-v in some versions of IE) is annoying.

 It's well worth the minor amount of toolbar space to have a button for a
 second dropdown bar. Even if it does cause user confusion, which I don't
 think will occur, as people are used to multiple toolbars (thanks to
 Word).

 The specific change required is to wp-includes/js/tiny_mce_config.php. The
 line that defines $mce_buttons is an array of strings.

 The string 'wp_adv_start' defines the split point between the normal
 toolbar and the advanced hidden toolbar. The string 'wp_adv' is the button
 to hide/unhide that bar. So having 'wp_adv' after 'wp_adv_start' makes
 little sense anyway, as this puts that button on the hidden toolbar, where
 it can never actually be clicked.

 So, proposed change:
 -Move 'wp_adv' directly before 'wp_adv_start'.
 -Eliminate 'separator' after 'wp_adv' and 'wp_adv_start' for looks only.

 This works very well in 2.1.3. Have not tested with 2.2.

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


More information about the wp-trac mailing list