The linkit button in TinyMCE doesn't work when in fullscreen. Upon research, I tracked this down to the fact that Drupal.wysiwyg.instances[editor.id].format on line 23 of editor_plugin.js is failing.

This is showing as undefined. Upon further research it is evident that the editor.id when in fullscreen is mce_fullscreen and this does not exist in the instances object. This causes the format to be undefined and fail.

I may have some time to work on this, but don't have a ton of TInyMCE experience for how to handle the seperate editor when in fullscreen.

If anyone has some input please let me know.

CommentFileSizeAuthor
#2 linkit-fullscreen-2782339-2-D7.patch1.7 KBccjjmartin
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

rhigginsME created an issue. See original summary.

ccjjmartin’s picture

ccjjmartin CreditAttribution: ccjjmartin as a volunteer commented
FileSize
linkit-fullscreen-2782339-2-D7.patch1.7 KB

I tested this on a Panopoly distribution site and will have some related issues in WYSIWYG and Panopoly that are all required to fix this fully. From a linkit only perspective this works but WYSIWYG currently has issues in fullscreen mode and Panopoly has !important set on the z-index which makes it appear like this patch is broken.

ccjjmartin’s picture

ccjjmartin CreditAttribution: ccjjmartin as a volunteer commented
Version: 7.x-3.4 » 7.x-3.x-dev
Status: Active » Needs review
ccjjmartin’s picture

ccjjmartin CreditAttribution: ccjjmartin as a volunteer commented
Related issues: +#2903753: Regression: TinyMCE fullscreen mode

  • anon committed e6780a7 on 7.x-3.x authored by ccjjmartin 
    Issue #2782339 by ccjjmartin: Linkit Button doesnt work with TinyMCE...
anon’s picture

anon
Primary language Swedish
Location Gothenburg
CreditAttribution: anon commented
Status: Needs review » Fixed

Thanks for reporting, patching.

Status: Fixed » Closed (fixed)

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