🇫🇷France @vanessa.fayard

vanessa.fayard → created an issue.

Hi,

The patch works for me (Drupal 11 - Gin 4.0), after clearing the cache.

Thanks for the issue and the patch!

Hi!

I didn't reproduce the error in 7.0.x

Hi!

I tested the patch on Drupal 10.4 & better_exposed_filters ^7.0

It works on an single exposed filter (equal, less or greater)

It doesn't work on a exposed filter “between” (min - max)

Patch suggestion

vanessa.fayard → created an issue.

Thanks for the feedback!

I've updated the MR. I've added the command documentation to the README.md.

Have a nice day!

Hello,
I have made an MR proposal. MR !10

I did not implement the interactive command because I encountered problems between version 12 and 13 of Drush. The ask() method changes drastically between these two versions.
So, I opted for a command with arguments and options to create the entity.

Example of command:

drush orejime:create-entity videos 'Youtube videos' --description="this is the description text" --publish --default

Un café, de la musique, et c'est parti !
Programme du jour : contrib module contrib, traduction...

vanessa.fayard → created an issue.

vanessa.fayard → created an issue.

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

I have the same result as on the before and after reference screenshots.

Tested on Drupal 10.3.10 'drupal/paragraphs:^1.18'

Thanks

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.