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.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Matroschker created an issue. See original summary.

anon’s picture

Status: Active » Closed (cannot reproduce)

I cannot reproduce this.

If you can, please proive screenshots and if you can, check the css properties in the dialog.

bekro’s picture

I'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

bekro’s picture

Status: Closed (cannot reproduce) » Active
BarisW’s picture

Version: 7.x-3.3 » 7.x-3.x-dev

Same 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.

VoidE’s picture

I 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.

VoidE’s picture

Status: Active » Needs review

ron_s’s picture

Status: Needs review » Needs work

Thanks 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.

anon’s picture

Status: Needs work » Postponed (maintainer needs more info)
ron_s’s picture

Status: Postponed (maintainer needs more info) » Needs work

That'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.

ron_s’s picture

Just 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.

ron_s’s picture

I found the problem. I'll post a patch.

The linkit.js file sets zIndex: 210000 in Drupal.linkit.modalOptions. However, linkit.css has this code in it:

.ui-dialog.ui-front.linkit-wrapper {
  z-index: 2000;
}

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.

anon’s picture

Yes 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?

anon’s picture

@ron_s: The lastest dev contains this css.

.ui-dialog.ui-front.linkit-wrapper {
  z-index: 210000;
}
ron_s’s picture

Status: Needs work » Closed (duplicate)
Related issues: +#2782339: Linkit Button doesn't work with TinyMCE when in full screen

Ah, 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.