🇫🇷France @vanessa.fayard

vanessa.fayard → made their first commit to this issue’s fork.

I've applied the MR!9 as patch and it applied successfully with no errors.

Tested on Drupal 10.3.2 with drupal/ckeditor_iframe: ^3.0@beta

Thanks

Patch update for D10 compatibility, from issue #3367901

Works on Drupal 10.2.7 - PHP8.2
Version : 1.0.2

Patch #17 - Verified and tested it on a Drupal 10.2.7 and PHP 8.2.19
The patch works fine.

Thanks

Hello,

Patches 1 and 2 successfully tested and validated.
Proposal 2 approved to enhance contrast and readability of active element.

Patch 1 by @david_nawrot

Patch 2 by @n.pineau

The patch works fine, thank you for adding a margin to the element.

While checking the patch, I noticed a problem of alignment in the case of adding only 1 type of paragraph.

css/paragraphs.claro.scss - 8

  // Add top margin to buttons to separate buttons,
  // because Claro theme sets that to 0.
   > .button {
      margin-top: 1rem;
  }

The margin-top should be set to . field-actions to handle all cases.
align-items: center is managed in issue 3417771

Verified and tested it on a Drupal 10.2.2 and paragraphs module 8.x-1.17.

before :

after :

The patch works fine.

May need to add padding/margin between button and text to improve UX.

Verified the #9 and tested it on a Drupal 10.2.2 and paragraphs module 8.x-1.17.
Followed the steps give by @emilymathew

Before :

After :

The patch works fine for me.