Dropbuttons and regular buttons are misaligned

I updated the margin values of .form-actions .action-link to match the 0.5rem margin used for .dropbutton-wrapper in the Claro theme's dropbutton.css file. This will ensure that the margins are consistent across both types of buttons.

We can even think of updating the margin values of the .dropbutton-wrapper CSS to match the 1rem margin used for .form-actions .action-link in the Claro theme's form.css file.

Verified the MR!3470 and tested it on Drupal version 9.5.x. The patch works fine and I have added the before and after screenshots for reference.

Change does seem to fix the issue.

Verified the merge request on #3, issue has been fixed.

Attached patch for 10.1.x

Verified the patch #9 and tested it on Drupal version 10.1.x. The patch works fine and I can confirm that the issue is resolved in Drupal 10 version also. Screenshots are same as in #5.

The Needs Review Queue Bot → tested this issue. It no longer applies to Drupal core. Therefore, this issue status is now "Needs work".

Apart from a re-roll or rebase, this issue may need more work to address feedback in the issue or MR comments. To progress an issue, incorporate this feedback as part of the process of updating the issue. This helps other contributors to know what is outstanding.

Consult the Drupal Contributor Guide → to find step-by-step guides for working with issues.

Patch ans issue version mismatch

Putting back to RTBC then.

Unrelated failure, restoring status

The last submitted patch, 9: 3341669-9-10.1.x.patch, failed testing. View results →

I have created this patch for 10.1.x-dev and its working as expected. Please verify

Applied #18 patch on Drupal 10.1.x-dev with Claro theme and It's working fine.

Supplying a 9.5.x version of the patch for anyone upgrading to Claro before the Drupal 10 upgrade.

Copied the fixing lines from MR!3470 of #3 and reapplied on Drupal 11.x.

Reviewed the code and tested the latest patch against 11.x, text is now aligned.

Nice work here! I posted a minor comment on the MR regarding duplicate properties.

Fixed the change requested by laurii.

Tested change and reviewed code, duplicate margin property removed.

Committed 33b9689 and pushed to 11.x. Also cherry-picked to 10.2.x. Thanks!

lauriii → closed merge request !5329

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