Nested paragraphs title gets cut off

Might be better to address this directly to the paragraphs module? As it might be an improvement for all users (?)

Moved the issue, cheers :)

Here is the patch fixing an issue.

Test failed unrelated.

Manual testing passes.

I thought perhaps the lines should be removed instead of setting to 'auto' but that didn't work, so the code looks good to me too.

I'm not 100% convinced the test suite fails are unrelated but I haven't gone digging so I'll split the difference and return it to NR.

I manually reviewed this and seems to work. Steps below are taken:

-Downloaded Drupal 9.5.10 and paragraph module (git 8.x-1-dev).
-Downloaded the patch on #14 and applied.
-Then complied the .scss and cleared cached
-See 2 attached, seems to show complete title for the nested paragraph

I tried the patch in #11, and I still see my paragraph type labels cut off to 100px. I think that is because of this rule:

.js  .paragraph-top {
    grid-template-columns: 100px auto 1fr auto; 
...

Is there any harm in using "auto auto 1fr auto" here instead?

Here is a patch with the change I proposed in my last comment. Not including an interdiff due to the simplicity.

Edit: and some changes seemed to be trivial differences from the compilation process.

After applying the patch from my last comment, I saw there was an additional spot that needed an update.

@bburg we have some activity in #3350521: Paragraph with long label name gets cut off issue which is a duplicate of this issue. I've closed that one but feedback in comment #3350521-14 still applies here. In short we can't change that grid width for first column from px to auto because it will mess with collapse summary rendering. Same comment has some ideas how to solve this better.

Hi @pivica, I just attempted to fix this issue as per your suggestions in the comment #3350521-14 and created the MR132. Please see if this is the expected solution. If this is fine, then I will refactor summary rules to use the new class. Also, attaching a screen recording for reference.

Hi @nayana_mvr thanks for update but unfortunately I can not open mov format on my Linux. Anyway I think for this kind of problems screenshots are anyway better for reviewing stuff. I will try to test your MR in the evening and do more detail test locally but what I see right now is that previous patch in this commit (for example comment #19) was doing some additional CSS changes and not just changing from 150px to auto. What are we going to do with those additional stuff?

Hi @pivica,
Yes, there are some additional CSS changes in the patch which I think is not really required for this issue. One is related to changing flex-basis: 100px; to flex-basis: auto; for the element .paragraph-type-title. But I couldn't find any such element in the paragraphs UI. I noticed that that change was part of the patch #11 which was not working as per comment #17.
The other change is related to .paragraphs-convert-button element which is again not available in the UI. Also, when I applied the patch in #19, that change was not getting applied. If possible, could you please check it from your end as well?
Attaching few screenshots also for reviewing.

Hello,

I tested the current MR with the latest released version of the module (8.x-1.19). This fixes the cut wording issue (when the paragraph is expanded), however I now have 4 warnings:

• Warning: Undefined array key "top" in paragraphs_field_widget_single_element_form_alter() (line 530 of modules/contrib/paragraphs/paragraphs.module).
• Warning: Trying to access array offset on null in paragraphs_field_widget_single_element_form_alter() (line 530 of modules/contrib/paragraphs/paragraphs.module).
• Warning: Trying to access array offset on null in paragraphs_field_widget_single_element_form_alter() (line 530 of modules/contrib/paragraphs/paragraphs.module).
• Warning: Trying to access array offset on null in paragraphs_field_widget_single_element_form_alter() (line 530 of modules/contrib/paragraphs/paragraphs.module).

The first condition is too permissive and is valid for my "moderation_state" field (which haven't a "top" key).

Here is my patch proposal.

I can create a fork and a merge request if it's good for you.

Hey recently had the same issue and decided to create a combined patch file until the issue is merged into the main branch.

Patch contains the following:

• initial patch from oleksandr p: #11
• MR changes from nayana_mvr : #23
• Fixes to MR from s3b0un3t #27

hopefully this helps!

Added a new patch based on alton307's one, but for 8.x-1.20

rerolling patch for latest of 1.x-dev with a small test coverage.

Drupal 11.4.x, paragraphs: 1.22.0