[wp-trac] [WordPress Trac] #44025: Privacy: Pagination screen options for the requests list tables

WordPress Trac noreply at wordpress.org
Tue Jul 10 22:42:44 UTC 2018


#44025: Privacy: Pagination screen options for the requests list tables
-------------------------------------------------+-------------------------
 Reporter:  birgire                              |       Owner:  (none)
     Type:  enhancement                          |      Status:  new
 Priority:  normal                               |   Milestone:  4.9.8
Component:  Privacy                              |     Version:
 Severity:  normal                               |  Resolution:
 Keywords:  has-screenshots has-patch needs-     |     Focuses:
  testing                                        |  administration
-------------------------------------------------+-------------------------

Comment (by pbiron):

 Replying to [comment:18 birgire]:
 > yes that's was my understanding too regarding the screen setup, just
 before the hook fires.

 thanx for the confirmation.

 > I think we can also remove the {{{'default' => 20}}} and use the
 fallback, like other tables.

 I thought about that as well, but then noticed that
 [[https://core.trac.wordpress.org/browser/trunk/src/wp-admin/edit.php#L292
 |/wp-admin/edit.php#:292]] and
 [[https://core.trac.wordpress.org/browser/trunk/src/wp-admin/edit-
 tags.php#L57|/wp-admin/edit-tags.php#L57]] explicitly set `'default' =>
 20`, so I left it in.  I can take it out if you feel strongly, but I
 suggest leaving it in.

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/44025#comment:19>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list