Thank you for reporting. I did a bit different solution. I changed the operators for default values in drupalSettings and the settings form.
This is now included in the 1.0.1 version.
Thank you for reporting it.
This is now solved in the 1.0.1 version.
I added the solution from this issue: https://www.drupal.org/project/ckeditor_tooltips/issues/3515056 🐛 Document that this module require jQuery Active
This is now solved in the 1.0.1 version.
Thank you for reporting it and creating the patch.
This is now included in the 1.0.1 version.
Thank you for reporting it and creating the patch.
This is now included in the 1.0.1 version.
Thank you for reporting it. I checked it. I changed it a bit.
yareckon, thank you for reporting it. The issue is the watch script, because it runs the --mode development
and if the last build was done with npm run watch
, the eval was in the JS file.
Currently there is no plan to add this feature in the near future, but I see it would be really useful.
If you don't want to go to the source code, you can select the text, delete it and write it back.
If someone has time to implemented, I will be happy to test it and commit it.
I am adding this as a note how to reproduce it:
This happens when use the tooltip on a word. It doesn't happen when you use the default blue circle with the letter "i". If you use it with the default blue circle you can just click it and hit backspace.
But only if I display all the items. When switching 50 per page, it's working.
Yes, exactly.
Escalating to Critical, because it breaks the website.
Sorry, I missed this task https://www.drupal.org/project/autologout/issues/3469258 🐛 Multiple dialogs open / have to click multiple times Needs review . Probably they are the same. I am marking it as a duplicate, but I can re-open it if it isn't so.
The #89 works on 10.4.1
Thank you @adamps. The patch from https://www.drupal.org/project/exif_orientation/issues/3477919 🌱 Overall status Active worked for me using the 8.x-1.x-dev version.
The patch worked for me for 8.x-1.7. Thank you.
@tasc good suggestion. I added the filter and moved the more_text and less_text to it.
This is available in 3.0.0-alpha3.
I am keeping this task open, because I think it would be good to put also the other data to the filter.
There was a problem with the usage of the variables in the Drupal.t() function, so I create the filter for that.
This is available in 3.0.0-alpha3.
@julienvey what did you did so that the string was visible in the interface translation? Or how did you translate it?
Thank you @Gmaheux.
This is now available in 3.0.0-alpha2
Available in 3.0.0-alpha2
This is a good suggestion for a new feature.
Currently the read more/less texts are defined in the CKEditor settings for the Text format and are the same everywhere. If we want to have different read more/less texts per element there is much more work to be done. The CKEditor5 plugin has to be rewritten to allow for editing the data-readmore-more-text and data-readmore-less-text per element.
I am keeping the issue open, because it would be a nice feature, but currently there is no plan to implement it. I am also adjusting the tile of the task, so that it reflects the proposed feature.
Relate issue https://www.drupal.org/project/ckeditor_readmore/issues/3225661 ✨ Configuration in field-values Active
@Gmaheux thank you for the fixed. I removed toggler.classList.add(classes.replace(/ /g, ","));
, because then the classes are duplicated. Is there a reason you added it?
Thank you. Will be added to the next release.
We're not using this module anymore, because it was too buggy. Instead we now use an external service.
I've used the #2 and created the merge request.
I tried the #2 on the 2.0.0-alpha5 version and it works.
As this is quite problematic if you refresh the page after you get the error, because it completely delete the node I agree with @andreasderijcke that this is a major issue. I am also changing the status back to RTBC.
Patch for 1.0.2
joco_sp → created an issue.
I don't know who is a tourist for you, but did you check the permissions for that role? There are many that can prevent non-admin users to use them.
Did you check if that user has the permission to use the Editable fields (see screenshot)? Or if the user can edit this entity? You also have the option to bypass the access for a dedicated field (see the screenshot).
I like the "editablefields-popup-edit-link", so I added it. Of course it can be changed to whatever the maintainers prefer :)
Example of the working widget. As I mentioned above, I had to change the "Select fallback display mode:" to "teaser".
I'm not sure what would be the best fix for the code. As a temporary solution I am creating a new display view for that mode with the wanted fields and selecting it as a fallback.
When debugging the EditableFieldsFieldFormatter.php on line 230, it looks like it's going into a loop and if I comment out this whole part, the node is shown with the editable field.
// Popup version.
if ($this->getSetting('behaviour') === 'popup') {}
I think that the problem is in the return, because it is returning the same field on the same display and therefore it goes into a loop. If I change this to "teaser" it works. Now just the "Edit" link is displayed twice when I save the form in the popup.
Thank you @andrew.wang, but in the meantime I found this module Keep referenced entities → .
Before re-applying for Prevent term delete I would like to try out Keep referenced entities, because if it will work as expected, it will be even better for our case. I like it more that it covers all entities. If it won't work then I'll come back to this module.
Thank you. It's added to 2.x. It will be available in the next release.
As @Anas_maw mentioned, this was solved in the parent issue.
Thank you all for the support.
I merged it with the 2.x branch and it will be available in the next release.
Instead of
$time = (int) $time;
I added
$time = (float) $time;
otherwise you are getting the read time of 0 min. Additionally we could show also seconds, but this would be a new feature.
Thank you @HeikkiY. I commited it to 2.x branch. It will be available in the next release.
It will be available in the next release.
@malcomio thank you for the support and wait. I used the solution from the related issue 🐛 I get an error when previewing a node. Fixed as it checks for the id sooner.
Quite an old issue :D Thank you for your support.
I committed it do 2.x. It will be available in the next release.
Thank you all for the support. I merged the patch #14. It will be available in the next release.
@cmconklin thank you for expressing the interest to co-maintain the module. I added you :)
@vishal.kadam thank you for the review.
I applied the fixes.
Thank you @gisle. I contacted both maintainers and applied for security advisory policy.
Thank you @andrew.wang. I opened a new one on the project's page Offering to maintain Prevent Term Delete 💬 Offering to maintain Prevent Term Delete Active .
Hi @cmconklin did you apply for speeding up the deployment of your issue ( https://www.drupal.org/project/ckeditor_readmore/issues/3388400 ✨ CKEditor 5 Support Fixed ) or did you want to make more contributions to the module?
Hello @apaderno.
Is there a chance that I can become the co-maintainer/maintainer of this module (
https://www.drupal.org/project/prevent_term_delete →
)? We are using it on some websites and for almost half a year we are running it with patches.
For the start I would like to make a Drupal 10 release and fix some other issues that are also tested and ready to be committed:
-
https://www.drupal.org/project/prevent_term_delete/issues/3354210
📌
Automated Drupal 10 compatibility fixes
RTBC
-
https://www.drupal.org/project/prevent_term_delete/issues/3157733 →
I also contacted karthikeyan-manivasagam, but as you didn't have any luck getting his response, I decided to write also here. In case this can speed up the process and make it possible.
Thank you. Then I am closing this as it is outdated.
@cossovich thank you. This functionality is now in 3.x
levmyshkin nice feature :)
As 2.x is deprecated, this has to be ported to 3.x