🇮🇳India @shobhit_juyal

shobhit_juyal → created an issue.

If we can just add a check to see if we are not editing default translation otherwise proceed for to add violation rule. Does this break any check ?

shobhit_juyal → made their first commit to this issue’s fork.

shobhit_juyal → made their first commit to this issue’s fork.

The changes made in the MR are sufficient to understand. closing this issue.

This issue came again in latest branch 8.x-2.5

I believe the issue raised by @gwvoigt is not actually related with the CKEDITOR5 compatibility, rather it should be open in a separate issue queue for the logical issue.
Please share the issue link if it has opened already.

Thanks

Yes, not working for me as well, followed @longwave suggestion to run it from top level.

Drupal version   : 10.0.0
PHPUnit 10.5.3 by Sebastian Bergmann and contributors.

Looks good, thanks for the commit.

Sharing the patch

shobhit_juyal → created an issue.

Adding new patch as further issues detected.

attaching patch

shobhit_juyal → created an issue.

Providing patch 3398770-giella_was_not_found-2.patch

shobhit_juyal → created an issue.

Hi,

I faced the same issue and added a patch for it. Kindly review.

The same issue was coming in bp-accordion and patch suggested in #22 is fixing that issue as well.

Patch is not applying on Drupal v. 10.1.0

Changing status to Active

3102249-port-for-D9.3.22-63.patch

Patch failed for Drupal 10.1.0

Patch is tested and working fine for me and #6. Drupal version - 10.1.0

Moving this to RTBC.

Thanks
mrcdrx

Hi,

Please correct my understanding, wheather this Drupal 10 compatibility fix should not also consider the dependecies in composer.json as well.
This module has a dependency on captcha module which already has compatible Drupal 10 branch but that is not updated in this module's info file.

The changes are working fine and fixed this issue:

I tested it on Drupal - 10.1.0-dev.

Hi @Chris64,

Further should we exclude drupal.message library as well ?

My apologies for the delay in responding. Your patience is appreciated.

Yes, you are right about the issue. The module installation was not properly done on that moment (might be some other issues) and after that I was facing that error. Later I thought that if it has occurred then it should be handled and hence I created this issue.

Please close this if it should.

shobhit_juyal → created an issue.

@bnjmnm, I apologize for any inconvenience caused.
In stark theme, if we can just add z-index says 1260 then it will also work. like this:
z-index: 1260;

and here its UI after that update.

Hope, if this can help.

shobhit_juyal → made their first commit to this issue’s fork.

shobhit_juyal → created an issue.

3047392-17 - Patch is skipping -