spree-contrib / spree_editor

Rich text editor for Spree with Image and File uploading in-place.
http://guides.spreecommerce.org
BSD 3-Clause "New" or "Revised" License
112 stars 183 forks source link

Editor is breaking in the view #36

Closed suryart closed 11 years ago

suryart commented 11 years ago

Hi,

Not sure what's the issue here. But, editor is kind of breaking the UI.

Please, see this image for your reference.

editor

Do let me know if it's a known issue? or there is anything I have broken at my end?

JDutil commented 11 years ago

The css styles need to be fixed. I've switched to CKEditor, and it does not have this issue.

robertmassaioli commented 11 years ago

This is happening for me too using the 1-3-stable branch of spree-editor. Ideally it would be fixed soon.

robertmassaioli commented 11 years ago

I worked out the problem here. It is twofold.

To see these two changes made to your configuration then see here:

diff --git a/config/tinymce.yml b/config/tinymce.yml
index da7c7aa..2cabe4d 100644
--- a/config/tinymce.yml
+++ b/config/tinymce.yml
@@ -2,11 +2,12 @@ mode : "exact"
 theme : "advanced"
 language : "<%= I18n.locale.to_s.split('-')[0].downcase %>"
 skin : "o2k7"
-width : "594"
+#width : "594"
 plugins : "safari,style,layer,table,advhr,advimage,inlinepopups,insertdatetime,preview,media,searchreplace,contextmenu,paste,directionality,fullscreen,noneditable,visualchars,nonbreaking,xhtmlxtras,template"

 # Theme options
-theme_advanced_buttons1 : "cut,copy,paste,pastetext,pasteword,|,search,replace,|,undo,redo,|,link,unlink,anchor,image,cleanup,help,code,|,insertdate,inserttime,preview,|,fullscreen,|,charmap,media,advhr,|,visualchars,blockquote"
+theme_advanced_buttons1 : "cut,copy,paste,pastetext,pasteword,|,search,replace,|,undo,redo,|,link,unlink,anchor,image,cleanup,help,code,|,insertdate,inserttime,preview"
+theme_advanced_buttons5 : "fullscreen,|,charmap,media,advhr,|,visualchars,blockquote"
 theme_advanced_buttons2 : "bold,italic,underline,strikethrough,|,justifyleft,justifycenter,justifyright,justifyfull,|,sub,sup,|,bullist,numlist,|,outdent,indent,blockquote,|,"
 theme_advanced_buttons3 : "styleselect,formatselect,fontselect,fontsizeselect,|,forecolor,backcolor"
 theme_advanced_buttons4 : "tablecontrols,|,hr,removeformat,visualaid"

This should solve this problem for everybody and I might even send this back as a PR in a minute.

JDutil commented 11 years ago

Has been fixed by @robertmassaioli