GrapesJS / grapesjs

Free and Open source Web Builder Framework. Next generation tool for building templates without coding
https://grapesjs.com
BSD 3-Clause "New" or "Revised" License
22.38k stars 4.06k forks source link

Related Links ? #335

Closed FlashPapa closed 7 years ago

FlashPapa commented 7 years ago

On the newletters demo, the linking of text does not work with the ckeditor plugin, the code is not updated. We are forced to click again on the text box to force the update of the source. There is the same problem with grapesjs-mjml

It is damage, that makes this beautiful program is unusable with basic things!

artf commented 7 years ago

Does this solve the issue?

document.querySelector('.cke_dialog').addEventListener('mousedown', function(e){ 
    e.stopPropagation();
})
FlashPapa commented 7 years ago

Actually it works, I have extended the problem on the modal backdrop and it is ok! Like this :

CKEDITOR.on('dialogDefinition', function(e) {
            $timeout(function() {
                document.querySelector('.cke_dialog_background_cover').addEventListener('mousedown', function(e) {
                    e.stopPropagation();
                });
                document.querySelector('.cke_dialog').addEventListener('mousedown', function(e) {
                    e.stopPropagation();
                });
            }, 300);
        });

Regards,

lock[bot] commented 5 years ago

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.