[wp-trac] Re: [WordPress Trac] #9606: screen_layout for plugins

WordPress Trac wp-trac at lists.automattic.com
Tue Apr 21 20:58:25 GMT 2009


#9606: screen_layout for plugins
-----------------------------+----------------------------------------------
 Reporter:  arena            |       Owner:  arena           
     Type:  feature request  |      Status:  new             
 Priority:  high             |   Milestone:  2.8             
Component:  Administration   |     Version:  2.8             
 Severity:  normal           |    Keywords:  has-patch tested
-----------------------------+----------------------------------------------

Comment(by arena):

 Replying to [comment:7 azaozz]:
 > Replying to [comment:6 Denis-de-Bernardy]:
 > > unless you build an editor for a different data type and write the
 needed scripts.
 > In this case the page would be mostly copied from the write posts page?
 Can probably use the same columns setting too.
 >
 > Actually the different settings for posts and pages should probably be
 merged too, don't see a used case where one would need 2 columns and the
 other only one as both screens are almost identical.
 >
 > The patch also wouldn't work if the user_option is not set (the
 "useless" check does exactly that).

 (the "useless" check does exactly that).

 sorry if i follow the code, in the switch, if the $screen is not found =>
 return ''
 so in the test :

 {{{
 if ( ! $screen_layout_columns )
 }}}

 $screen_layout_columns is always true...

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/9606#comment:9>
WordPress Trac <http://trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list