fix(datalist): actions wrapper by tlabaj · Pull Request #10939 · patternfly/patternfly-react

@andrew-ronaldson @lboehling just want to run this change by you and make sure this design change is OK. I created a codepen showing a regular data-list and one with the proposed change - https://codepen.io/mcoker/pen/PorgJab

Just as a reminder, the TL;DR of this change is to update all of the "action" cells in a data-list to have a negative top/bottom margin that is equal to a button's top/bottom padding. We already do this for plain actions (kebabs in the codepen), and this will do the same for non-plain actions. What that effectively does is keeps the presence of a button in a row from increasing the height of the row, and it aligns the button text with the data-list text.

On a row with a single line of text, this looks great IMO. It vertically centers the button in the row.

Before
Screenshot 2024-09-12 at 8 09 59 PM

After
Screenshot 2024-09-12 at 8 09 50 PM

But when text wraps and the row is taller, it's more obvious that non-plain actions are shifted up. I just want to confirm that's ok from y'alls perspective:

Before
Screenshot 2024-09-12 at 8 14 43 PM

After
Screenshot 2024-09-12 at 8 14 49 PM