There is an annoying issue in IE11 (and maybe in other versions of IE). In panels popup CKEditor replaces textarea, however it's invisible. No JavaScript errors are reported.

CommentFileSizeAuthor
ckeditor_panels_ie11.png22.82 KBwwalc

Comments

loopduplicate’s picture

Confirmed to be broken in IE11. Works fine in IE9 and IE10.

loopduplicate’s picture

I'm getting closer to solving this one. Here's the latest progress.

There is a line of code that gets injected <style>.cke{visibility:hidden;}</style> in the head. There is another line of CSS that gets loaded after that in editor.js, .cke_chrome{visibility:inherit}. For some reason, in IE11, the cascading is backwards and .cke takes precedence over .cke_chrome.

There is a similar thread here: https://drupal.org/node/2078821

loopduplicate’s picture

I was able to get CKEditor to show up using:

.cke.cke_chrome {
 visibility: inherit;
}

but dropdowns didn't work, as reported on https://drupal.org/comment/8680039#comment-8680039 . Then I added the following to make the dropdowns work:

.cke.cke_panel {
  visibility: visible;
}

Here's the snippet in its entirety:

// IE11 fix. See https://drupal.org/node/2244835 and https://drupal.org/node/2078821
.cke.cke_chrome {
 visibility: inherit;
}
.cke.cke_panel {
  visibility: visible;
}

I'm not sure if this is a bug that should be addressed in CKEditor itself or one that should be addressed perhaps here, in this contrib module. It's getting late tonight and I have to make it to DrupalCamp Stanford tomorrow so I don't have time to figure that out right now :) If anyone has a chance to test this stuff in CKEditor outside of Drupal, that would be helpful. In the meantime, I hope that people can use my little fix in their theme for the time being.

Cheers,
Jeff

loopduplicate’s picture

Status: Active » Needs work

Marking this as needs work, even though I don't have a patch ready, because the code I used works but I don't know exactly where the right place is to inject it :)

  • Commit fa98ab8 on 7.x-1.x authored by loopduplicate, committed by wwalc:
    Issue #2244835 by loopduplicate: CKEditor is not visible in Panels IPE...
wwalc’s picture

Status: Needs work » Fixed

Thanks loopduplicate, I've added the styles to ckeditor.css. I've spent some time on trying to reduce the test case so that I could report a generic ticket for CKEditor that does not require Drupal with aditional modules, but had to give up. Considering that we're hitting some edge case, I'm ok with having it solved this way until it is identified as a bug in CKEditor library itself and fixed there.

Regarding the proper place for such styles, there is a separate issue for this: #1370894: Conditionally load ckeditor.css only when necessary
I hope to have it closed soon as well.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.