- ๐บ๐ธUnited States chrissnyder Maryland
Is this still an issue with the latest versions of Drupal? Is this reproducible without other CKEditor modules enabled?
- ๐จ๐ฆCanada mgifford Ottawa, Ontario
This is still a good idea.
As a start we could include it in the CKEditor Toolbar configuration admin/config/content/formats
-
mark_fullmer โ
committed fa333b32 on 7.x
Issue #3381766 by carma03, mark_fullmer, berdir: Linkit Dialog list has...
-
mark_fullmer โ
committed fa333b32 on 7.x
- ๐บ๐ธUnited States mark_fullmer Tucson
As of Drupal 10.5 and Drupal 11.2, version 45 of CKEditor is used, which places the Editor Advanced Link items in its own collapsible fieldset within the CKEditor Link editor. Given this, it is is likely that this proposed change is outdated. I'll not close the issue just yet, but I'm going to set its status to "Postponed" to evaluate the impact of version 45 of CKEditor.
- ๐ฎ๐ณIndia shubhamgoel Delhi
Hi,
Created a merge request to resolve the UI Issue. Please review and merge the MR.
- @shubhamgoel opened merge request.
- First commit to issue fork.
- ๐บ๐ธUnited States mark_fullmer Tucson
mark_fullmer โ changed the visibility of the branch 3381766-linkit-dialog-list to hidden.
- ๐บ๐ธUnited States mark_fullmer Tucson
I've confirmed that the recent series of patches do not correctly handle both scenarios that need to be supported. They fix the dialog when used in CKEditor, but break the dialog when it is associated with a Link field, as was indicated by berdir in #20.
I've created a new MR, https://git.drupalcode.org/project/linkit/-/merge_requests/113 , which scopes the absolute positioning to the Link field by excluding that CSS from the CKEditor context. Can others confirm that this resolves the display issues for them in CKEditor? (It does for me, but as has been observed in this discussion, part of the issue has to do with the order in which CSS files are loaded, which may differ depending on site configuration.)
- @mark_fullmer opened merge request.
I was having the same issue, and the patch #25 โ fixed it:
Drupal core 10.5.1
Linkit 7.0.5Before:
After applying the patch #25:
- ๐ฏ๐ดJordan Rajab Natshah Jordan
Thanks for the MR and the static patch file.
Hoping that this issue get committed to Drupal Core - @man-1982 opened merge request.