Hello,
First of all, I would like to thank you for the realization of the module which is very interesting.

I installed it in its version 8.x-1.0, and I notice that the buttons within the modal are not defined as primary buttons. As a result this displays the horizontal bar below the button and makes it difficult to read the text. I suggest you add the "button--primary" class to each button.
I attach a before
Before Update

and after
After Update

screenshots.

Sincerely
Marc

CommentFileSizeAuthor
#2 improve-readibility-add-buttons-3266847-2.patch715 bytesmartygraphie
after_update_displayed_button.png40.2 KBmartygraphie
before_update_displayed_button.png38.91 KBmartygraphie

Issue fork paragraphs_browser-3266847

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

Martygraphie created an issue. See original summary.

martygraphie’s picture

martygraphie commented
StatusFileSize
new improve-readibility-add-buttons-3266847-2.patch715 bytes

I add the patch to correct the problem

phannphong’s picture

phannphong commented

I can confirm that the patch #2 is good. But the patch is fixed only for "Add" button without fixing overlap if we have many paragraphs (at least we have more than 1 line) in the modal. It's referenced in the issue CSS overlap in D10 Claro

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
commented
Status: Active » Needs review

Since there's a patch it should be in needs review status.

@phannphong do you feel this patch and the D10 Claro path work well together?

phannphong’s picture

phannphong commented

@greggles, yes, it works well. You can check my screenshot.
https://prnt.sc/24a1Kij7E0w7

damienmckenna made their first commit to this issue’s fork.

damienmckenna opened merge request !26

damienmckenna changed the visibility of the branch 8.x-1.x to hidden.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented

It might be worth verifying the correct class to use with the Gin theme.