Skip to content

Abilities Explorer: Remove cb column.#246

Merged
jeffpaul merged 4 commits intoWordPress:developfrom
juanfra:tweak/abilities-remove-cb-column
Feb 23, 2026
Merged

Abilities Explorer: Remove cb column.#246
jeffpaul merged 4 commits intoWordPress:developfrom
juanfra:tweak/abilities-remove-cb-column

Conversation

@juanfra
Copy link
Member

@juanfra juanfra commented Feb 20, 2026

What?

Removes the unused "cb" (checkbox) column from the Abilities Explorer table and deprecates the related method.

Why?

The "cb" column is intended for bulk actions, which we don't have in the Abilities Explorer. Having it there is confusing and clutters the UI.

How?

Removed the "cb" column from the table display and deprecated the public method that handled it (to avoid breaking changes if extended elsewhere).

Testing Instructions

  1. Visit the "Abilities explorer".
  2. Confirm the checkbox column is not there.

Screenshots or screencast

Before After
Screenshot 2026-02-20 at 18-25-14 Abilities Explorer ‹ Stories® — WordPress Screenshot 2026-02-20 at 18-24-53 Abilities Explorer ‹ Stories® — WordPress
Open WordPress Playground Preview
@github-actions
Copy link

github-actions bot commented Feb 20, 2026

The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the props-bot label.

If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message.

Co-authored-by: juanfra <juanfra@git.wordpress.org>
Co-authored-by: dkotter <dkotter@git.wordpress.org>
Co-authored-by: jeffpaul <jeffpaul@git.wordpress.org>

To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook.

@codecov
Copy link

codecov bot commented Feb 20, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 55.75%. Comparing base (77819fb) to head (f14c507).
⚠️ Report is 15 commits behind head on develop.

Additional details and impacted files
@@              Coverage Diff              @@
##             develop     #246      +/-   ##
=============================================
- Coverage      55.84%   55.75%   -0.09%     
+ Complexity       508      507       -1     
=============================================
  Files             32       32              
  Lines           2611     2606       -5     
=============================================
- Hits            1458     1453       -5     
  Misses          1153     1153              
Flag Coverage Δ
unit 55.75% <ø> (-0.09%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.
@jeffpaul
Copy link
Member

Yeah, I suppose that was there with the assumption there was some sort of bulk actions which are not present so fine by me to remove as long as @dkotter is happy with the technical changes here to proceed on approve/merge

@jeffpaul jeffpaul assigned jeffpaul and juanfra and unassigned jeffpaul Feb 23, 2026
@jeffpaul jeffpaul requested a review from dkotter February 23, 2026 16:46
@jeffpaul jeffpaul added this to the 0.4.0 milestone Feb 23, 2026
@jeffpaul jeffpaul moved this to Needs review in WordPress AI Planning & Roadmap Feb 23, 2026
'<input type="checkbox" name="abilities[]" value="%s" />',
esc_attr( $item['slug'] )
);
_deprecated_function( __FUNCTION__, 'x.x.x', 'The checkbox column is no longer displayed.' );
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we need to deprecate this? I understand that's the typical best approach but in this case:

  1. Seems unlikely anyone is overriding this
  2. But if they are, this method exists on the WP_List_Table class we're extending so if we remove this method entirely (instead of deprecating) anyone extending this won't run into problems (I don't think)
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the review!

  1. Seems unlikely anyone is overriding this
  2. But if they are, this method exists on the WP_List_Table class we're extending so if we remove this method entirely (instead of deprecating) anyone extending this won't run into problems (I don't think).

Fair point, I went this route because it's a public method, and even though the chance of someone overriding it is pretty minimal, seemed appropriate to be a bit defensive.

Totally fine with removing it completely though :)

@jeffpaul jeffpaul merged commit 32a1d80 into WordPress:develop Feb 23, 2026
16 of 22 checks passed
@github-project-automation github-project-automation bot moved this from Needs review to Done in WordPress AI Planning & Roadmap Feb 23, 2026
@juanfra juanfra deleted the tweak/abilities-remove-cb-column branch February 23, 2026 22:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants