@berdir I just tested version 1.18 with Claro and it definitely looks better than it was before. Not ideal, unfortunately, because we still have issue with empty table cells (see screenshot below), but it doesn't catch your eye.
Hi guys,
during my work on multiple projects where paragraphs approach was used to build content, I also noticed that paragraphs widget became slow. And here is the solution I used and am using on current projects.
The main problem is metatag module and its form, which crazy slows down user interface: https://www.drupal.org/project/metatag/issues/3057523 π Content forms are painfully slow if multiple sub-modules are enabled Needs work , especially if you use Schema.org submodules. While performance issue is not solved, you absolutely have to use https://www.drupal.org/project/metatag_async_widget β which loads metatags form asynchronously on demand. This small module improved the speed of loading single paragraph for editing from 20s to 0.4s (!).
Another nice thing, which doesn't affect performance as much as previous module, but still good to use, is already mentioned https://www.drupal.org/project/paragraphs_ee β which moves paragraph selection in modal window.
I strongly suggest using these two modules to improve performance and provide better UI for paragraphs.
Hey @gΓ‘bor-hojtsy, sorry for late response, I have just received notification.
>Why are you getting a different result?
Well, I guess that it is due the fact that I had this issue few months back (oct 2023), probably new version of the dependency arrived (as I actually expected in the ticket description), and conflict was resolved for PHP 8.2. I see that you replaced MathieuViossat\Util\ArrayToTextTable with dekor\ArrayToTextTable library to also support PHP 8.3 and I think this is great solution. Thank you very much for taking care!
GΓ‘bor Hojtsy β credited nsavitsky β .
Ouch, this is painful. The moving actions to new column obviously crashed functional testing. I'll try to find some time to update tests if my fix makes sense.
I forgot to add patch itself. Here it goes.
@hlopes Thank you very much for this patch and description, it helped me so much to understand what happened with paragraphs widget after 10.2 update.
So, if I understand correctly, in 10.2 we have "very special" new column for having actions. It looks like the problem is more complex than the case with cardinality = 1, because with 1.x paragraphs widget shows empty table cell (which actually breaks css/js) and shows actions in the content/title cell. I guess the correct approach will be to actually use this new cell, instead of hiding. That's why I extended the patch of @hlopes to implement this solution.
It works great with Claro.
Before patch:
After patch:
Unfortunately, it doesn't look good with Gin, because it shows paragraphs widget using "display:block" and has some magic css for calculating width and I don't really understand how it works (but it was broken before patch too, so it doesn't make it any worse :) ).
Okay, I guess I will ease the pain for someone like me who needs to update old project with existing content made by Color API/JQuery Colorpicker fields to Drupal 10. Here is how to make it works using composer.
First, until this issue is merged, you need to say composer that you want to use code from merge request as the source. Modify repositories section in composer.json like this.
"repositories": [
{
"type": "composer",
"url": "https://packages.drupal.org/8",
"exclude": [
"drupal/jquery_colorpicker"
]
},
{
"type": "composer",
"url": "https://asset-packagist.org"
},
{
"type": "package",
"package": {
"name": "drupal/jquery_colorpicker",
"version": "dev-3310798-implement-a-subscriber",
"type": "drupal-module",
"source": {
"url": "https://git.drupalcode.org/issue/jquery_colorpicker-3310798.git",
"type": "git",
"reference": "3310798-implement-a-subscriber"
}
}
}
]
Now, run "composer require drupal/jquery_colorpicker:dev-3310798-implement-a-subscriber". Composer will install the module using the source code from merge request #7.
Next, you need to add JQuery Colorpicker vendor library. For some reason it was not installed in my case on previous step. Run "composer require jaypan/jquery-colorpicker:~1.0.1".
Almost there. Last step is to apply patch from https://www.drupal.org/project/vendor_stream_wrapper/issues/3325235 β which helps Vendor Stream Library to handle relative URLs from CSS.
"drupal/vendor_stream_wrapper": {
"https://www.drupal.org/i/3325235": "https://git.drupalcode.org/project/vendor_stream_wrapper/-/merge_requests/6/diffs.patch"
}
And ta-da! Tested on Drupal 10.1.5 and PHP 8.1.
nsavitsky β created an issue.
Updated patch for 6x.
@abramm This is just wow. First, this patch saved me a lot of time and, second, this patch is just masterclass. Thank you so much. I have updated your patch for 8.x.-6.x and it works just perfect.
nsavitsky β created an issue.
nsavitsky β created an issue.