[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