Skip to content
This repository has been archived by the owner on Jul 14, 2022. It is now read-only.

TinyMCE 4 compatibility #36

Closed
pomartel opened this issue Jun 25, 2013 · 21 comments
Closed

TinyMCE 4 compatibility #36

pomartel opened this issue Jun 25, 2013 · 21 comments

Comments

@pomartel
Copy link
Contributor

Hi @PerfectlyNormal ! I was just wondering if you had any plans to make the plugin compatible with the latest TinyMCE 4 release. I haven't found any good alternatives so far that works with Rails. Let me know!

@PerfectlyNormal
Copy link
Owner

Hi.

I haven't paid any attention to tinyMCE 4. Had a quick look now (only have my phone), and the editor looks very nice.

I'm definitely interested in getting it working. Will probably be a few days before I can try it out though.

Sent from my iPhone

On 25 jun 2013, at 11:12 em, Pierre Olivier Martel notifications@github.com wrote:

Hi @PerfectlyNormal ! I was just wondering if you had any plans to make the plugin compatible with the latest TinyMCE 4 release. I haven't found any good alternatives so far that works with Rails. Let me know!


Reply to this email directly or view it on GitHub.

@pomartel
Copy link
Contributor Author

Great! I played with the new TinyMCE today and it is in fact a lot nicer and faster. The UI is also much more clear. I would in fact suggest using the new layout as a base for the plugin. If you look at their example there and click on the image button, the box is simple and I guess you could reuse that for the plugin. Let me know if I can help in any ways.

@pomartel
Copy link
Contributor Author

Do you have a donation link for the project? I would like to show you my support and encourage you to get the plugin working with tinyMCE 4 as soon as possible ! :)

@PerfectlyNormal
Copy link
Owner

Hi.

I don't have any kind of donations set up, no. Didn't really believe anyone would bother with that.
I do however have an Amazon wish list 1 I keep for when christmas gets close and relatives wonder what to buy, if you really want to. It contains mostly books, and Kindle editions work very well for most of them.

However, a donation/gift/bribe isn't really necessary, and I was meaning to give it a go sometime this weekend anyways.

Best regards,
Per Christian B. Viken

On Jun 29, 2013, at 4:19 PM, Pierre Olivier Martel notifications@github.com wrote:

Do you have a donation link for the project? I would like to show you my support and encourage you to get the plugin working with tinyMCE 4 as soon as possible ! :)


Reply to this email directly or view it on GitHub.

@PerfectlyNormal
Copy link
Owner

Well, I have started work on this now.

I did, as I often do, decide to do a little too much at the same time, underestimating how much work it actually is both getting it to work fine with TinyMCE4 and rewriting and restructuring everything, so it's all a hopeless mess that doesn't even work right now.

I hope I'll get some free time tomorrow as well, so I can get it at least partially working, and push it up for experimentation. There's probably a ton of things that could be done to make it fit better with the architecture, and I'll study the existing plugins for ideas.

@pomartel
Copy link
Contributor Author

Haha no worries! I know the feeling trying to get out of a hole you dug too deep! I'll be glad to test it out and give you my opinions once you have something working.

@PerfectlyNormal
Copy link
Owner

I've pushed my changes now.
Things left to do:

  • It doesn't seem to actually submit the values in the form
  • Dialog misses styling
  • Spinner (throbber) doesn't work (dialog needs to get the TinyMCE theme CSS)
  • Should see if it's possible to drop dialog.html and use the new way of creating dialogs (had trouble getting a <input type='file'> in there)
  • Icon needs to be improved (Icon should be a vector image #37)
  • Should add the action to the menu bar as well as the button bar
  • Extensive testing in IE8+, Firefox, Safari, Chrome and others we might care about

PerfectlyNormal added a commit that referenced this issue Jul 1, 2013
@pomartel
Copy link
Contributor Author

pomartel commented Jul 2, 2013

Great start! I'll fork it this week and see if I can at least help on the CSS part. Let me know if you make other improvements until then.

@pomartel
Copy link
Contributor Author

I worked a lot on this and banged my head against the wall since there is so little documentation about the TinyMCE 4 API. I tried really hard to make the plugin work and in the end I changed my approach to use the file_browser_callback instead to develop my own file browser that integrates with the image plugin. It's working good although there are still a lot of little glitches that I can't fix without the proper hooks in the TinyMCE api.

I won't release this as a gem since there is a lot of business logic in there and it uploads directly to S3. So unfortunately I won't be able further help you because I moved away from your gem but if you have any questions regarding TinyMCE 4 API, I'm getting pretty good at it after reading a lot of the source code!

@PerfectlyNormal
Copy link
Owner

Hi.

Yes, the documentation could be a lot better. Wasn't easy figuring out everything, and there's still quite a bit left for this gem to be usable.
I'm sorry to see you leave, as you've been a great help so far.

We use the gem at my work as well, and I'm planning to get us upgraded to TinyMCE4 in the near future, so I might be able to sneak in an hour or three fixing the gem while at work. I've had a vacation for the last few weeks now, and that's mostly why I haven't had too much motivation to get things going, and development have been going (too) slow.

Anyways, glad you got a solution that works for you.

Best regards,
Per Christian B. Viken

On Jul 18, 2013, at 10:52 PM, Pierre Olivier Martel notifications@github.com wrote:

I worked a lot on this and banged my head against the wall since there is so little documentation about the TinyMCE 4 API. I tried really hard to make the plugin work and in the end I changed my approach to use the file_browser_callback instead to develop my own file browser that integrates with the image plugin. It's working good although there are still a lot of little glitches that I can't fix without the proper hooks in the TinyMCE api.

I won't release this as a gem since there is a lot of business logic in there and it uploads directly to S3. So unfortunately I won't be able further help you because I moved away from your gem but if you have any questions regarding TinyMCE 4 API, I'm getting pretty good at it after reading a lot of the source code!


Reply to this email directly or view it on GitHub.

@benjaminwood
Copy link

Hey Per,

Thanks for your work on this gem! I've got a project that is currently using this with tinymce v3 and they'd like to upgrade to v4. Do you still have plans of rewriting this?

Thanks again!

@PerfectlyNormal
Copy link
Owner

Hmm, I replied to the mail, but doesn't look like it worked… or maybe it'll just appear in a while.
Anyways,

I'm glad you like it.

Yes, I'm still planning on getting a version that's usable with v4 finished sometime soon, I've just really been swamped with work, and too tired to do much in the evenings.
Our main app is using v3 a lot, and I'd really like to get us to v4, but it's not exactly a priority, since v3 works fine.
I think I might have an hour or three to spare this weekend, so I'll try to do something.

Nagging usually helps, 'cause that makes me feel bad for neglecting the project ;)

@PerfectlyNormal
Copy link
Owner

Hi.

I'm glad you like it.

Yes, I'm still planning on getting a version that's usable with v4 finished sometime soon, I've just really been swamped with work, and too tired to do much in the evenings.
Our main app is using v3 a lot, and I'd really like to get us to v4, but it's not exactly a priority, since v3 works fine.
I think I might have an hour or three to spare this weekend, so I'll try to do something.

Nagging usually helps, 'cause that makes me feel bad for neglecting the project ;)

Best regards,
Per Christian B. Viken

On Aug 23, 2013, at 5:27 PM, Benjamin Wood notifications@github.com wrote:

Hey Per,

Thanks for your work on this gem! I've got a project that is currently using this with tinymce v3 and they'd like to upgrade to v4. Do you still have plans of rewriting this?

Thanks again!


Reply to this email directly or view it on GitHub.

@benjaminwood
Copy link

That's great, thanks for your reply and your work on this.

In the mean time, I actually managed to hack it together with a custom upload handler! :-)

@nicholasmartin
Copy link

My rails app is using Tinymce 4 cloud solution, and this would have been PERFECT for my project! I really hope you get this working soon Per !

Benjamin, could you share your solution in the mean time please?

@benjaminwood
Copy link

@lifestylenetworker I'd share if it was helpful, but our solution ended up being really custom to the app. It's not really worth recreating elsewhere. Sorry!

@nicholasmartin
Copy link

No problem, thanks

Nick Martin
Let's ROCK!!!

This email is very short since it's crafted on my iphone.

On Thu, Sep 12, 2013 at 4:07 AM, Benjamin Wood notifications@github.com
wrote:

@lifestylenetworker I'd share if it was helpful, but our solution ended up being really custom to the app. It's not really worth recreating elsewhere. Sorry!

Reply to this email directly or view it on GitHub:
#36 (comment)

@jalcine
Copy link

jalcine commented Dec 29, 2013

Curious about what's the progress on this. I have time to help out; I'd just like to know how I can @PerfectlyNormal.

@jalcine
Copy link

jalcine commented Dec 29, 2013

Is that checklist still up to date?

@PerfectlyNormal
Copy link
Owner

Hi, and thanks for your interest.

There's not been any progress lately. We are still using TinyMCE3 a few places at work, with no hurry to upgrade to 4, and when I attempted it last, it had changed a lot, and things wasn't very cooperative, causing me to lose my motivation, and then I've just been silently ignoring my duties for a long while.

Any help would be appreciated. I think the checklist is up to date. Might be something more to do, but I don't think any of those unchecked have been taken care of yet.

@PerfectlyNormal
Copy link
Owner

In case anyone is still paying attention, I finally got off my ass and started fixing this again. Tracking the progress in #50

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants