Fix DataViews primary actions horizontal layout consistency

[priethor]

What?

Always add the HStack wrapper to dataviews actions.

Why?

Right now, the horizontal wrapper is not added when there are only primary actions, which results in actions:

• Stacking vertically.
• Being always shown instead of only when hovering/selecting the row

How?

Adding the HStack unconditionally, and conditionally adding the dropdown menu.

Testing Instructions

1. Before applying this patch, hack /packages/edit-site/src/components/post-list/index.js to only show primary actions with const actions = useMemo(() => [ editAction, ...postTypeActions ].filter( action => action.isPrimary )...);
2. Go to the post list experiment, verify that all the actions show and are vertically stacked.
3. Apply this PR
4. Go to the post list experiment, verify that all actions show in a row and only when hovering/selecting.

Screenshots or screencast

Before	After

[github-actions]

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: priethor <[email protected]>
Co-authored-by: youknowriad <[email protected]>
Co-authored-by: mcsf <[email protected]>
Co-authored-by: oandregal <[email protected]>

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

[mcsf]

Tests well, makes sense. Thanks for this.

[oandregal]

Being always shown instead of only when hovering/selecting the row

I believe this was on purpose, otherwise it looks like there's nothing to interact with when there's only primary actions (see "after" screenshot). Before shipping, I'd welcome @jameskoster thoughts on this. Do we make the primary actions always visible when there are no secondary ones? or do we only display them on hover/select?

[priethor]

it looks like there's nothing to interact with

That's a good point; note that the primary reason for this PR is preventing the vertical stacking, so it would make sense to show them without the need to hover.

[oandregal]

note that the primary reason for this PR is preventing the vertical stacking, so it would make sense to show them without the need to hover.

Feel free to 🚢 when you're ready if the actions visibility works as in trunk. That change is a separate concern. I just wanted to double-check with design about that to avoid going back/forth every few months on this topic.

[github-actions]

Flaky tests detected in 5b7b9ff.
Some tests passed with failed attempts. The failures may not be related to this commit but are still reported for visibility. See the documentation for more information.

🔍 Workflow run URL: https://github.com/WordPress/gutenberg/actions/runs/17102335240
📝 Reported issues:

• [Flaky Test] ensures List View global shortcut works properly #51533 in /test/e2e/specs/editor/various/list-view.spec.js

[priethor]

I missed the coauthoring, I owe you one 🤦 🙇

[jameskoster]

@oandregal good question. I think it's okay for now, and loosely similar to wp-admin (where actions only appear on hover/focus). But as DataViews are consumed more broadly across the new admin design we may need to revisit actions appearance and behaviors.