🇦🇲Armenia @arthur.baghdasar

arthur.baghdasar → created an issue.

arthur.baghdasar → changed the visibility of the branch 3514288-make-visual-adjustment to hidden.

arthur.baghdasar → created an issue.

I dont see the changes in the PR

This is included in the latest 2.0.x release

arthur.baghdasar → created an issue.

arthur.baghdasar → changed the visibility of the branch 3429689-automated-drupal-11 to hidden.

The Sortable js need to get in order for my code to work.

arthur.baghdasar → created an issue.

Same issue for us

New patch which fixes scrolling. In some cases when there is a section which opens a nested modal the scroll position is lost.

Fixed with Sortable JS implementation

This is now working for me and ready for testing.

arthur.baghdasar → created an issue.

arthur.baghdasar → created an issue.

Added a case to handle the redirected paths as well.

Updated the tests.

Works for me

arthur.baghdasar → created an issue.

Works for me.

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

Thanks

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

Fixed.

arthur.baghdasar → created an issue.

arthur.baghdasar → created an issue.

See the fix here
https://www.drupal.org/project/ckeditor5_fullscreen/issues/3381084#comme... 🐛 Fullscreen mode interferes with link button Needs review

Created a fix for this.

I've experienced the same, but this happens when admin_toolbar module is enabled

Works for me

arthur.baghdasar → created an issue.

arthur.baghdasar → created an issue.

arthur.baghdasar → created an issue.

I've created plugin for CKEDITOR 5

Included in the latest 2.0.1 release. thanks

Tagged a new 2.0.0 release, which includes this change.

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

Doesnt work with nested paragraph required fields.

Not working for CKeditor5

Merged, will be included in the next release, thank you

Rerolled

Ive removed this part from the patch everything seems to be working fine for me.
In the code below $input variable comes with an Alias and I don't understand why we should get the $path from it.
In My case the $path variable is converted back to the node/[nid] which is then being set to be the new input.

+    if (!empty($input) && \Drupal::moduleHandler()->moduleExists('path_alias')) {
+      /** @var \Drupal\path_alias\AliasManagerInterface $aliasManager */
+      $aliasManager = \Drupal::service('path_alias.manager');
+      $path = $aliasManager->getPathByAlias($input);
+      if ($path !== $input) {
+        $input = $path;
+      }
+    }

Welcome, thanks for the interest

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

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

Rerolled patch

Rerolled patch for 2.x branch

Closing this, I don't see the issue with the latest stable version.

I do see the issue mentioned in the #135 regarding the required fields.