[wp-trac] [WordPress Trac] #18082: Reset screen options button

WordPress Trac wp-trac at lists.automattic.com
Wed Aug 17 03:02:54 UTC 2011


#18082: Reset screen options button
-----------------------------------------------+---------------------------
 Reporter:  danielbachhuber                    |       Owner:
     Type:  enhancement                        |  danielbachhuber
 Priority:  normal                             |      Status:  accepted
Component:  Administration                     |   Milestone:  Awaiting
 Severity:  minor                              |  Review
 Keywords:  ux-feedback has-patch 2nd-opinion  |     Version:
                                               |  Resolution:
-----------------------------------------------+---------------------------

Comment (by nacin):

 I would keep the if() logic in set_screen_options() and
 reset_screen_options(), and consider combining your reset_screen_options()
 with set_screen_options() -- it's nice and easy to have a single POST
 handler here.

 Patch looks alright at a glance, but there's a lot moving around, so we'd
 have to look closer.

 There should also be a hook so a plugin can reset theirs too.

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


More information about the wp-trac mailing list