-
Notifications
You must be signed in to change notification settings - Fork 19
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Feature Request: load tinymce from assets.. #24
Comments
Hi @bluscreen Good idea 👍 |
Hi marucjmar any progress on this? otherwise I guess I'll have to manually install tinymce via node 🤕 |
in case youre still interested, i got an approach running now using npm and ember browserify. its still quite brittle but it works.. |
I'm working on it. You can see my progress on branch |
Hi thats probably due to minification and or fingerprinting. You can
disable this in ember cli build config. Cheers Daniel
Marcin Lazar <notifications@github.com> schrieb am Mo., 26. März 2018,
09:32:
… I'm working on it. You can see my progress on branch load-from-assets. In
dev environment work correctly, but not when build production app.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#24 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AFPtJZcmbnsSUxgmqtd_FOKj4hzXTkJVks5tiJmmgaJpZM4R_BF1>
.
|
++ also waiting for this |
Hi @marucjmar, Could you roll this out? ember-cli-tinymce is always using tinymce version 4.4 for me and that has a bug with image attachments that's affecting me and is fixed in later versions (link to issue). Thanks for the contribution! |
@ziali088 can you show your environment.js config?
|
Yes updating the config does fix it, but my concern was more of a now buggy version being the default :) |
Hi marucjmar
I would very much prefer to load the tinymce locally instead of cdn.tinymce.com
Do you think you can integrate this feature?
Cheers
Daniel
The text was updated successfully, but these errors were encountered: