[wp-trac] [WordPress Trac] #42066: List tables: consider a new method to generate the views links markup

WordPress Trac noreply at wordpress.org
Tue Jul 12 01:27:27 UTC 2022


#42066: List tables: consider a new method to generate the views links markup
-------------------------------------------------+-------------------------
 Reporter:  afercia                              |       Owner:  costdev
     Type:  enhancement                          |      Status:  assigned
 Priority:  normal                               |   Milestone:  6.1
Component:  Administration                       |     Version:
 Severity:  normal                               |  Resolution:
 Keywords:  has-patch needs-testing has-unit-    |     Focuses:
  tests dev-feedback                             |
-------------------------------------------------+-------------------------

Comment (by costdev):

 Thanks for the feedback @davidbaumwald and @afercia, and thanks
 @hellofromTonya for milestoning this one!

 I have updated [https://github.com/WordPress/wordpress-develop/pull/2828
 PR 2828] to:

 - Rename `get_admin_status_links` to `get_views_links`.
 - Update all references within child classes in the PR.
 - Abstract the `current` HTML markup as well. The `current` key is now
 `bool`.
 - Update the tests.

 I also tested some of the plugins in
 [https://wpdirectory.net/search/01G7QZWB07E3114X6W4ZNT4ZBG this result]
 and [https://wpdirectory.net/search/01G7R00R7YM9HV9Y47DTZZHQFD this
 result], including:
   - WordPress San-Thumbnail Posts Plugin
   - BuddyPress
   - Media Library Assistant
   - Contact Form to Manage and respond to conversations with customers —
 Happyforms
   - ATUM WooCommerce Inventory Management and Stock Tracking

 All of the plugins' views links seem to appear and function as expected.
 The PR is ready for review.

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


More information about the wp-trac mailing list