Links do not align properly in Views UI field/sort rearrange dialog

Here's a patch that adds a few lines to core/themes/claro/css/components/views-ui.css.

Oops, forgot to include the changes in the accompanying .pcss.

Confirmed the issue in the IS.
Changed to a minor though.
Applied patch #5 and it did fix it.

The last submitted patch, 5: 3376159-4.patch, failed testing. View results →

Believe to be random. Restarted tests

Ideally we could use '#wrapper_attributes' but we can't because it's also picked up by the table column 😬 Maybe something like this could work?

Tested patch #9 and verified on the content view links are still aligned with the Remove table header.

I'm triaging RTBC issues → . I read the IS and the comments. I didn't find any unanswered questions or other work to do.

Leaving at RTBC.

The last submitted patch, 9: 3376159-9.patch, failed testing. View results →

Seems unrelated

The last submitted patch, 9: 3376159-9.patch, failed testing. View results →

Hi,

As per ticket status (Needs Work) looks the issue, but found it's resolved by the #9 patch. resolved the Remove links alignment issue.

After patch:

Aditi Saraf → made their first commit to this issue’s fork.

Hi , I have fixed the issue . Please review .

The last submitted patch, 17: 3376159-17.patch, failed testing. View results →

@Aditi Saraf there is no interdiff or explanation for why the solution has changed. #9 still applies and was moved to NW by a random failure. Hiding #17 and restoring RtBC status

The Needs Review Queue Bot → tested this issue.

While you are making the above changes, we recommend that you convert this patch to a merge request → . Merge requests are preferred over patches. Be sure to hide the old patch files as well. (Converting an issue to a merge request without other contributions to the issue will not receive credit.)

Hi all,

I applied and tested patch #9 in drupal 11, Issue is resolved.
Patch # 17 is not required.

Testing steps:

1. Install the latest version of Drupal
2. Go to Structure -> Views -> "Comments" (for example)
3. Click the dropdown icon next to Fields - Add and click Rearrange

Before

After

Moving this to RTBC

Thakns!

DrupalCI is broken, and the tests on #9 are failing. Can someone convert #9 to an MR and hide the patches? It can be immediately marked RTBC if the tests pass.

smustgrave → changed the visibility of the branch 3376159-links-do-not to hidden.

Hiding patches and dangling branch

Committed and pushed 59f0e738c0 to 11.x and 2774aab1e0 to 10.2.x. Thanks!

Automatically closed - issue fixed for 2 weeks with no activity.