Skip to content

Update the custom post list UI to look like the Posts screen#25300

Merged
crazytonyli merged 4 commits intotrunkfrom
task/custom-post-list-row
Feb 25, 2026
Merged

Update the custom post list UI to look like the Posts screen#25300
crazytonyli merged 4 commits intotrunkfrom
task/custom-post-list-row

Conversation

@crazytonyli
Copy link
Contributor

Description

This PR also adds a feature flag to show Posts and Pages that's backed by wordpress-rs. I plan to show them on XMLRPC disabled sites, which will be done separately.

Unlike the regular posts, the custom posts list now shows an "All" tab, which is similar to the wp-admin.

@crazytonyli crazytonyli added this to the 26.8 milestone Feb 25, 2026
@crazytonyli crazytonyli requested a review from kean February 25, 2026 09:19
@dangermattic
Copy link
Collaborator

1 Warning
⚠️ View files have been modified, but no screenshot or video is included in the pull request. Consider adding some for clarity.

Generated by 🚫 Danger

@sonarqubecloud
Copy link

@wpmobilebot
Copy link
Contributor

App Icon📲 You can test the changes from this Pull Request in WordPress by scanning the QR code below to install the corresponding build.
App NameWordPress
ConfigurationRelease-Alpha
Build Number31123
VersionPR #25300
Bundle IDorg.wordpress.alpha
Commitf7425ed
Installation URL53ndaa1ll6slg
Automatticians: You can use our internal self-serve MC tool to give yourself access to those builds if needed.

@wpmobilebot
Copy link
Contributor

App Icon📲 You can test the changes from this Pull Request in Jetpack by scanning the QR code below to install the corresponding build.
App NameJetpack
ConfigurationRelease-Alpha
Build Number31123
VersionPR #25300
Bundle IDcom.jetpack.alpha
Commitf7425ed
Installation URL5e1jp4iqaul80
Automatticians: You can use our internal self-serve MC tool to give yourself access to those builds if needed.

@wpmobilebot
Copy link
Contributor

🤖 Build Failure Analysis

This build has failures. Claude has analyzed them - check the build annotations for details.

Copy link
Contributor

@kean kean left a comment

Choose a reason for hiding this comment

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

LGTM. Left one comment.

return nil
}
.sorted { $0.slug < $1.slug }
.sorted(using: KeyPathComparator(\.name))
Copy link
Contributor

Choose a reason for hiding this comment

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

Just checking – name instead of slug – is that intentional?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, it is. During my test, I noticed that some post type slugs do not quite match their names. The order here determines how the CPTs are presented on the list. I thought we should order by their visible name, instead of their invisible slug.

@crazytonyli crazytonyli added this pull request to the merge queue Feb 25, 2026
Merged via the queue into trunk with commit 2329f73 Feb 25, 2026
29 of 34 checks passed
@crazytonyli crazytonyli deleted the task/custom-post-list-row branch February 25, 2026 21:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants