Skip to content
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

Modularizing the content editor #130

Closed
wants to merge 1 commit into from
Closed

Modularizing the content editor #130

wants to merge 1 commit into from

Conversation

chalmagean
Copy link
Contributor

This should alow to deface the current editor and use a third party
one like monologue_ckeditor.

This should alow to deface the current editor and use a third party
one like monologue_ckeditor.
@msevestre
Copy link
Collaborator

I would actually go a step further and replace the old editor with ckeditor
altogether .
@jp: what's your take on that?
On Feb 10, 2013 6:29 AM, "Cezar Halmagean" notifications@github.com wrote:

This should alow to deface the current editor and use a third party

one like monologue_ckeditor.

You can merge this Pull Request by running

git pull https://github.com/mixandgo/monologue master

Or view, comment on, or merge it at:

#130
Commit Summary

  • Modularize the content editor.

File Changes

Patch Links:

@jipiboily
Copy link
Owner

@msevestre: 👍

I would also prefer to see Tinymce replaced by ckeditor or any other editor that we might want to use for image upload. @mixandgo, are you planning on making image upload working? If yes, I would prefer to have CKEditor directly in Monologue with image uploading feature.

What are your thoughts on that? :)

/ref: #61

@msevestre
Copy link
Collaborator

I have just pushed a branch that does exactly that
https://github.com/msevestre/monologue/tree/ckeditor

I have used the work from @mixando and basically put in monologue itself

Not sure how to configure ckeditor . I copied the config over but their
might be a better way to simply change some features without copying the
whole file

What do you guys think?

On Mon, Feb 11, 2013 at 7:55 PM, Jean-Philippe Boily <
notifications@github.com> wrote:

@msevestre https://github.com/msevestre: [image: 👍]

I would also prefer to see Tinymce replaced by ckeditor or any other
editor that we might want to use for image upload. @mixandgohttps://github.com/mixandgo,
are you planning on making image upload working? If yes, I would prefer to
have CKEditor directly in Monologue with image uploading feature.

What are your thoughts on that? :)

/ref: #61 #61


Reply to this email directly or view it on GitHubhttps://github.com//pull/130#issuecomment-13396075.

@chalmagean
Copy link
Contributor Author

It might be too much to include it in monologue do to the fact that it generates it's own model and stuff. Some people might prefer it to be lighter, especially if they have say tinymce or other editor in a different place/gem.

What I would do is remove tinymce and provide it as an optional gem. Same goes for ckeditor.

@msevestre
Copy link
Collaborator

That's a good point.
That said, ckeditor provides a lot of UI by defaults for image upload. We
should probably use it for the image upload feature. That would be that
the image update feature would only be available for people using ckeditor?
I wonder if we are not making our life too complicated.....

On Mon, Feb 11, 2013 at 8:03 PM, Cezar Halmagean
notifications@git.luolix.topwrote:

It might be too much to include it in monologue do to the fact that it
generates it's own model and stuff. Some people might prefer it to be
lighter, especially if they have say tinymce or other editor in a different
place/gem.

What I would do is remove tinymce and provide it as an optional gem. Same
goes for ckeditor.


Reply to this email directly or view it on GitHubhttps://github.com//pull/130#issuecomment-13396507.

@chalmagean
Copy link
Contributor Author

Not sure if everyone needs the images and if they do, they might not like the ckeditor.

@jipiboily
Copy link
Owner

I must admit I like the idea of extracting the editor.

We could provide different options, one could be the "official" editor but having the choice of what you include in your app is a great thing. Especially when you mix multiple engines. This was core to Monologue's idea of having the least possible dependencies to ease it's use with other engines. I suffered a year ago using Spree and Refinery together (I know it's easier now :P) and this is how it all started in fact.

It looks like the 'ckeditor' gem supports image uploading, have you guys tried that feature? I did not try that gem at all yet; I am in crunch mode right now and barely have time to comment the issues here and answer mails...;)

cc'ing some Monologue users to have their inputs on that before we take a decision...

/cc @iwada @kwbock @TheEmpty @swalkinshaw @jvortmann @avitus: your inputs would be appreciated here. Would that be a good thing or a bad thing for your use case(s) to extract the editor from Monologue's core and provide different options via gems?

PS: here is another editor you could use for yourselves: https://github.com/jipiboily/monologue-markdown.

@chalmagean
Copy link
Contributor Author

I am working on making the upload work. I don't have too much free time but I'm working on it.

@chalmagean
Copy link
Contributor Author

Image upload seems to work fine, I've just tested it.

@msevestre
Copy link
Collaborator

Awesome!
do you have a branch that I could use and try?

On Fri, Mar 1, 2013 at 11:55 PM, Cezar Halmagean
notifications@git.luolix.topwrote:

Image upload seems to work fine, I've just tested it.


Reply to this email directly or view it on GitHubhttps://github.com//pull/130#issuecomment-14316274
.

@chalmagean
Copy link
Contributor Author

I can push the test app.

--- Cezar

On 02.03.2013, at 00:59, Michael Sevestre notifications@github.com wrote:

Awesome!
do you have a branch that I could use and try?

On Fri, Mar 1, 2013 at 11:55 PM, Cezar Halmagean
notifications@git.luolix.topwrote:

Image upload seems to work fine, I've just tested it.


Reply to this email directly or view it on GitHubhttps://github.com//pull/130#issuecomment-14316274
.


Reply to this email directly or view it on GitHub.

@msevestre
Copy link
Collaborator

cool. Let me know where to find it

On Sat, Mar 2, 2013 at 12:08 AM, Cezar Halmagean
notifications@git.luolix.topwrote:

I can push the test app.

--- Cezar

On 02.03.2013, at 00:59, Michael Sevestre notifications@github.com
wrote:

Awesome!
do you have a branch that I could use and try?

On Fri, Mar 1, 2013 at 11:55 PM, Cezar Halmagean
notifications@git.luolix.topwrote:

Image upload seems to work fine, I've just tested it.


Reply to this email directly or view it on GitHub<
https://github.com/jipiboily/monologue/pull/130#issuecomment-14316274>
.


Reply to this email directly or view it on GitHub.


Reply to this email directly or view it on GitHubhttps://github.com//pull/130#issuecomment-14316743
.

@chalmagean
Copy link
Contributor Author

@jipiboily
Copy link
Owner

I tried to add https://github.com/mixandgo/monologue_ckeditor to a dummy Monologue project that I have locally. I've got a few issues:

I ran:

rails generate ckeditor:install --orm=active_record --backend=carrierwave
rake db:migrate

Added that in the Gemfile:

gem "monologue_ckeditor", github: 'mixandgo/monologue_ckeditor'

Did I miss something?

@iwada
Copy link

iwada commented Mar 7, 2013

just to add to what @jipiboily observed, if you paste the Picture's url into the url field, the picture gets displayed, otherwise, http://cl.ly/image/1D2q1i170k1c

@avitus
Copy link
Contributor

avitus commented Mar 21, 2013

Sorry for the slow answer. I don't think extracting the editor would be too useful. I think most people are looking for a relatively simple, easy to install, lightweight blog engine. Given that the editor isn't too much of a differentiating feature, my guess would be that just picking one and sticking with it would be easiest.

I've used ckeditor in the past and have been very happy with it. But these days the functionality in all the editors is fairly comparable. My guess is that many people using Monologue are building it into a website for fairly non-technical users who just need the basic functionality.

(Just to add: I'm not opposed to extracting the editor, just don't think it would add too much)

@jipiboily
Copy link
Owner

Never too late @avitus ;)

The more I think to that, I must say that I would prefer to have a default editor that, if possible, would be easily replaceable if necessary.

If we finally extract it, I would make sure to have a Monologue installer and it would add to the Gemfile Monologue AND the "default"/official editor.

@msevestre
Copy link
Collaborator

@mixandgo , @iwada
ckeditor undefined methodkey?' for nil:NilClass`
I have the same issue. Did you guys manage to fix it?

@chalmagean
Copy link
Contributor Author

Sorry for the late reply, can you describe what you're doing so I can try and reproduce your issue ? I'm mostly using upload to server when adding images.

EDIT: You can add issues to the https://github.com/mixandgo/monologue_ckeditor repo

@chalmagean
Copy link
Contributor Author

I've updated https://github.com/mixandgo/blogtest and tested the upload to server functionality whichi seems to be working, I've also updated the readme. Please use that to test and report bugs so we all have the same setup.

@msevestre
Copy link
Collaborator

I see you are using carrierwave. I tried with paperclip. All right i try
again today and get back to you. Thanks for the feedback Cezar!
On Apr 13, 2013 6:10 AM, "Cezar Halmagean" notifications@github.com wrote:

I've updated https://github.com/mixandgo/blogtest and tested the upload
to server functionality whichi seems to be working, I've also updated the
readme. Please use that to test and report bugs so we all have the same
setup.


Reply to this email directly or view it on GitHubhttps://github.com//pull/130#issuecomment-16330673
.

@msevestre
Copy link
Collaborator

Here is what I am suggesting:

  1. We replace the default editor tinymce with CKEditor. CKeditor can be used without the image upload feature
  2. We create an extension that enables the Image upload using carrierwave. No need to use deface to replace the editor etc. Just need to copy the model files and run the migration. With carrierwave, I managed to get the upload working as well.

In the extension, we could set a new property that would specify where the pictures should be saved on the server

@jipiboily @mixandgo @avitus What do you guys think? I can submit a first PR next week doing that

@jipiboily
Copy link
Owner

That plan makes sense to me! :)

@msevestre
Copy link
Collaborator

This is no longer required as CKEditor is now the default editor for monologue.
Next step will be to add image management. Thanks for all your feedback guys

@msevestre msevestre closed this May 9, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants