Display node title by default when creating link with drupalink, allow configuration.

This issue needs tests.

At the moment this patch is not supporting multilingual sites. I would add the possibility to get the lable in the current language...

+    $langcode = \Drupal::service('language_manager')->getCurrentLanguage()->getId();
+    if ($entity->hasTranslation($langcode)) {
+      $trans = $entity->getTranslation($langcode);
+      $default_text = $trans->label();
+    } else{
      $default_text = $entity->label();
+    }
    // Allow other modules to alter default text.
    \Drupal::moduleHandler()

After analyzing, Below $attributes variables are always coming null even if using node url. $attributes["data-entity-uuid"] and $attributes["data-entity-type"] .

Patch in #2 works for us (we don't have a multilingual site). It boggles my mind that this behavior still isn't part of core.

Attached is the patch for #7 (multilingual sites).

Don´t know why the test fails. Patch is working fine for me.

BR,
Daniel

last try...

again

Updated

cspell fails for this, let's fix that as well. Steps to reproduce this locally are in https://www.drupal.org/pift-ci-job/1991994

Tried fixing the failed test case.
From core version 8.4.0 Drupal uses ES6 for JavaScript development.

Would be nice if we can have this feature with the next release :-/
One of the last core modifications we have at the moment...

Patch 2961554-19.patch is working fine on Drupal 9.2.2

Tests failing

Re-rolled patch #21, Fixed cs errors. Please review.

The Needs Review Queue Bot tested this issue. It either no longer applies to Drupal core, or fails the Drupal core commit checks. Therefore, this issue status is now "Needs work".

Apart from a re-roll or rebase, this issue may need more work to address feedback in the issue or MR comments. To progress an issue, incorporate this feedback as part of the process of updating the issue. This helps other contributors to know what is outstanding.

Consult the Drupal Contributor Guide to find step-by-step guides for working with issues.

#26 Suddenly stopped working I guess with some Core updates - will try to figure out why

Because CKEditor is moved to Contrib now I believe.

Have patched the same JS files here in new issue: https://www.drupal.org/project/ckeditor/issues/3340081

Re-roll for Drupal 9.5.*

Re-rolled patch for 9.5.x

Re-rolled patch for 9.5.x, based on #34 targeting the error "Missing parameter comment"

Re-rolled patch for 9.5.x, based on #35 targeting the error "fatal: corrupt patch at line 122"

Re-rolled patch for 9.5.x, based on #36 targeting the error "fatal: corrupt patch at line 121"

The Needs Review Queue Bot tested this issue. It no longer applies to Drupal core. Therefore, this issue status is now "Needs work".

Apart from a re-roll or rebase, this issue may need more work to address feedback in the issue or MR comments. To progress an issue, incorporate this feedback as part of the process of updating the issue. This helps other contributors to know what is outstanding.

Consult the Drupal Contributor Guide to find step-by-step guides for working with issues.

Fixed CCF from #34

The fix looks on the right lines, but tests needed.

With D10/CKEditor5: I needed the existing functionality that this was achieving, but I decided to use Linkit module as it seemed to be the easiest to use to do this: #3388565: Populate displayedText with entity title when left blank