frappe_tinymce
frappe_tinymce copied to clipboard
shridarpatil
TinyMCE Breaks Frappe In-App Email Functionality
Describe the bug This plugin is great, thanks so much for making it available! In my experience, it delivers on it's promise to load the TinyMCE editor instead of Quill in the standard text editor fields. Unfortunately, it simultaneously breaks the in-doc Email functionality in Frappe.
To Reproduce Steps to reproduce the behavior:
- Install frappe_tinymce
- Open any document.
- Scroll down to the bottom and select "New Email"
- See error in the browser console.
Expected behavior "New Email" should trigger a modal that allows the user to initiate a new email from the platform.
Actual behavior Nothing happens! The action just triggers an error in the console.
Additional context This was tested and triaged in both Firefox and Chrome browsers, on Frappe version 13 and 14 beta.
Here's the browser console error:
Uncaught TypeError: can't access property "open", o is undefined
tinymce.min.js:9:333666
And full browser console error log: erp.avu.nu-1648925803208.log
Hey! I saw this issue listed on Bounty source. Is this still open to work upon?
Hey there. So the thing is, I currently do not possess a Linux system😅 I would've love to work on this but for now I would not be able to complete it. Thanks.
No need for a Linux system in the age of WSL. But if that's not a great option, I'd be glad to provide a vps instance for this if you have the skill set otherwise.
It also breaks any Doctype which contains a text input in the quick create menu. For example, creating a new Issue doesn't work anymore as well.
I'm guessing tinyMCE basically doesn't load properly in any pop up. The built in email functionality is pretty useful for sending quotes, purchase orders and other routine documentation, and tinyMCE is great for writing long documents.
