Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Drupal 7.41, CK Editor 4.5.6, Linkit 7.x-3.5, Wysiwyg 7.x-2.2+63-dev
In maximized mode of CK Editor the linkit dialog appears in the background, if leaving maximized mode the dialog will be visible.
In the normal (not maximized mode) all works as expected.
Comments
Comment #2
anonI cannot reproduce this.
If you can, please proive screenshots and if you can, check the css properties in the dialog.
Comment #3
bekro CreditAttribution: bekro commentedI've had the same problem using version 7.x-3.3. I fixed it by setting the z-index of the maximized CK-Editor to 100 on creating the Linkit-Modal
Comment #4
bekro CreditAttribution: bekro commentedComment #5
BarisW CreditAttribution: BarisW commentedSame here, even with the latest DEV.
The problem occurs when the plugin is used in Ckeditor module (I didn't test it with the WYSIWYG Ckeditor integration.
Comment #6
VoidE CreditAttribution: VoidE commentedI found out the problem is caused by some versions of jQuery UI. In these versions, the z-index is removed form the modal options. I wrote a simple patch to re-apply the z-index on the linkt wrapper. I rather increase the z-index of the modal than decrease the z-index of the full-screen CKEditor like other patches do.
Comment #7
VoidE CreditAttribution: VoidE commentedComment #9
ron_s CreditAttribution: ron_s commentedThanks for the patch. Does not work with CKEditor 4.8.0 and jQuery 1.8. Same result as not having the patch applied -- the dialog is hidden when CKEditor is expanded. The dialog only is shown once the CKEditor window is no longer maximized.
Comment #10
anonI think #2782339: Linkit Button doesn't work with TinyMCE when in full screen solves this issue, right?
Comment #11
ron_s CreditAttribution: ron_s commentedThat's a fix for TinyMCE. This issue is for CKEditor.
Are you suggesting something similar needs to be created for CKEditor? Would still need a patch.
Comment #12
ron_s CreditAttribution: ron_s commentedJust did a quick test and the editor ID for CKEditor does not change when in fullscreen. The suggested patch on #2782339 will not solve the problem.
Comment #13
ron_s CreditAttribution: ron_s commentedI found the problem. I'll post a patch.
The linkit.js file sets
zIndex: 210000
inDrupal.linkit.modalOptions
. However, linkit.css has this code in it:This overrides the value set in the JS file.
All CKEditor rich text editors have a z-index of 9995 when in fullscreen mode, so a z-index of 2000 will always sit behind it.
Comment #14
anonYes its for TinyMCE but that z-index fix in that patch solved this for me when testing.
@ron_s: Are you using some other version of jQuery_ui as mentioned in #6?
EDIT:
Are you using the ckeditor module or wysiwyg with ckeditor?
Comment #15
anon@ron_s: The lastest dev contains this css.
Comment #16
ron_s CreditAttribution: ron_s commentedAh, I looked at the #2782339 patch and completely missed the CSS at the top. Was focused on the JS.
That fixes the problem. Using the CKEditor module, not WYSIWYG.