[PLIP-Advisories] Re: [Plone] #9249: Add TinyMCE as the default visual editor
#9249: Add TinyMCE as the default visual editor --+- Reporter: limi |Owner: robgietema Type: PLIP | Status: closed Priority: minor |Milestone: 4.0 Component: Visual Editor (Kupu) | Resolution: fixed Keywords:| --+- Changes (by robgietema): * status: assigned = closed * resolution: = fixed Comment: This plip has been merged. -- Ticket URL: http://dev.plone.org/plone/ticket/9249#comment:40 Plone http://plone.org Plone Content Management System ___ PLIP-Advisories mailing list plip-advisor...@lists.plone.org http://lists.plone.org/mailman/listinfo/plip-advisories
[PLIP-Advisories] Re: [Plone] #9249: Add TinyMCE as the default visual editor
#9249: Add TinyMCE as the default visual editor
--+-
Reporter: limi |Owner: robgietema
Type: PLIP | Status: assigned
Priority: minor |Milestone: 4.0
Component: Visual Editor (Kupu) | Resolution:
Keywords:|
--+-
Comment(by tom_gross):
The following error occurs when exporting the TinyMCE (1.1rc5)-profile via
portal_setup:
{{{
Traceback (innermost last):
Module ZPublisher.Publish, line 119, in publish
Module ZPublisher.mapply, line 88, in mapply
Module ZPublisher.Publish, line 42, in call_object
Module Products.GenericSetup.tool, line 656, in
manage_exportSelectedSteps
Module Products.GenericSetup.tool, line 1114, in _doRunExportSteps
Module Products.TinyMCE.exportimport, line 198, in exportTinyMCESettings
Module Products.GenericSetup.utils, line 809, in exportObjects
Module Products.GenericSetup.utils, line 530, in _exportBody
Module Products.TinyMCE.exportimport, line 128, in _exportNode
AttributeError: 'NoneType' object has no attribute 'split'
}}}
--
Ticket URL: http://dev.plone.org/plone/ticket/9249#comment:37
Plone http://plone.org
Plone Content Management System
___
PLIP-Advisories mailing list
plip-advisor...@lists.plone.org
http://lists.plone.org/mailman/listinfo/plip-advisories
[PLIP-Advisories] Re: [Plone] #9249: Add TinyMCE as the default visual editor
#9249: Add TinyMCE as the default visual editor --+- Reporter: limi |Owner: robgietema Type: PLIP | Status: assigned Priority: minor |Milestone: 4.0 Component: Visual Editor (Kupu) | Resolution: Keywords:| --+- Comment(by esteele): This PLIP has been accepted for merging into Plone 4.0 The final vote was: Alec Mitchell +1 David Glick +.5 Erik Rose +1 Laurence Rowe +1 Matthew Wilkes +1 Ross Patterson +1 Please merge your branches into the Plone 4.0 head by end-of-day Friday Oct 16. If you need assistance with merging, please contact me. We'll be assigning a documentation ticket to this PLIP shortly. Please assist the docs team in documenting the changes and new features that this PLIP introduces. -- Ticket URL: http://dev.plone.org/plone/ticket/9249#comment:38 Plone http://plone.org Plone Content Management System ___ PLIP-Advisories mailing list plip-advisor...@lists.plone.org http://lists.plone.org/mailman/listinfo/plip-advisories
[PLIP-Advisories] Re: [Plone] #9249: Add TinyMCE as the default visual editor
#9249: Add TinyMCE as the default visual editor --+- Reporter: limi |Owner: robgietema Type: PLIP | Status: assigned Priority: minor |Milestone: 4.0 Component: Visual Editor (Kupu) | Resolution: Keywords:| --+- Comment(by erikrose): For what it's worth It's worth a +1. :-) -- Ticket URL: http://dev.plone.org/plone/ticket/9249#comment:36 Plone http://plone.org Plone Content Management System ___ PLIP-Advisories mailing list plip-advisor...@lists.plone.org http://lists.plone.org/mailman/listinfo/plip-advisories
[PLIP-Advisories] Re: [Plone] #9249: Add TinyMCE as the default visual editor
#9249: Add TinyMCE as the default visual editor --+- Reporter: limi |Owner: robgietema Type: PLIP | Status: assigned Priority: minor |Milestone: 4.0 Component: Visual Editor (Kupu) | Resolution: Keywords:| --+- Comment(by erikrose): In OmniWeb 5.10.1, Safari 4.0.3, and Firefox 3.5.3, once I add any Discreet text, all the text I add afterward also ends up discreet, even if I switch to a different style or select `(remove style)`. The only way out is editing the raw HTML. This makes it a step backward from Kupu, from a user's point of view, imo. Once we fix this (and I suspect it's a bug in our use of it, as I wasn't able to reproduce it an TinyMCE's demo site), I'll happily change my vote to +1. Reproduction: 1. Edit the front page, and clear out the rich text field. 1. Type hello on line 1. 1. Assign the Discreet style to it. 1. Press Return, and type heading on line 2. 1. Select `Heading` from the `Style` menu. The Discreet style doesn't go away, but it should. No matter how you thrash about with `(remove style)` and such, you can't get rid of the `discreet` class on the heading without editing the raw HTML. -- Ticket URL: http://dev.plone.org/plone/ticket/9249#comment:33 Plone http://plone.org Plone Content Management System ___ PLIP-Advisories mailing list plip-advisor...@lists.plone.org http://lists.plone.org/mailman/listinfo/plip-advisories
[PLIP-Advisories] Re: [Plone] #9249: Add TinyMCE as the default visual editor
#9249: Add TinyMCE as the default visual editor --+- Reporter: limi |Owner: robgietema Type: PLIP | Status: assigned Priority: minor |Milestone: 4.0 Component: Visual Editor (Kupu) | Resolution: Keywords:| --+- Comment(by esteele): Erik, For what it's worth, I was able to fix this quickly by changing the discreet definition in the configlet to use a span instead of p tag. The remove formatting method by default only rips out span,b,strong,em,i,font,u, and strike elements (see http://dev.plone.org/collective/browser/Products.TinyMCE/trunk/Products/TinyMCE/skins/tinymce/tiny_mce.js#8787). I can see several solutions to this and they're all trivial, IMO. -- Ticket URL: http://dev.plone.org/plone/ticket/9249#comment:34 Plone http://plone.org Plone Content Management System ___ PLIP-Advisories mailing list plip-advisor...@lists.plone.org http://lists.plone.org/mailman/listinfo/plip-advisories
[PLIP-Advisories] Re: [Plone] #9249: Add TinyMCE as the default visual editor
#9249: Add TinyMCE as the default visual editor --+- Reporter: limi |Owner: robgietema Type: PLIP | Status: assigned Priority: minor |Milestone: 4.0 Component: Visual Editor (Kupu) | Resolution: Keywords:| --+- Comment(by rossp): I just noticed under a Plone 3.3.1 site that TinyMCE works for the static portlet add form, but not the edit form. I tried testing under the plip cfg in coredev but I can't add portlets at all on a newly created plone site. It seems there aren't any portlet managers/columns. Can someone test that TinyMCE works when editing an existing static portlet? -- Ticket URL: http://dev.plone.org/plone/ticket/9249#comment:31 Plone http://plone.org Plone Content Management System ___ PLIP-Advisories mailing list plip-advisor...@lists.plone.org http://lists.plone.org/mailman/listinfo/plip-advisories
[PLIP-Advisories] Re: [Plone] #9249: Add TinyMCE as the default visual editor
#9249: Add TinyMCE as the default visual editor --+- Reporter: limi |Owner: robgietema Type: PLIP | Status: assigned Priority: minor |Milestone: 4.0 Component: Visual Editor (Kupu) | Resolution: Keywords:| --+- Comment(by optilude): We use TinyMCE on a 3.3 site and it works fine, both to add and edit static text portlets. I also tested that in the PLIP bundle when I reviewed it a few weeks ago, and it worked. -- Ticket URL: http://dev.plone.org/plone/ticket/9249#comment:32 Plone http://plone.org Plone Content Management System ___ PLIP-Advisories mailing list plip-advisor...@lists.plone.org http://lists.plone.org/mailman/listinfo/plip-advisories
[PLIP-Advisories] Re: [Plone] #9249: Add TinyMCE as the default visual editor
#9249: Add TinyMCE as the default visual editor --+- Reporter: limi |Owner: robgietema Type: PLIP | Status: assigned Priority: minor |Milestone: 4.0 Component: Visual Editor (Kupu) | Resolution: Keywords:| --+- Comment(by davisagli): (In [29627]) finished FWT review, refs #9249 -- Ticket URL: http://dev.plone.org/plone/ticket/9249#comment:24 Plone http://plone.org Plone Content Management System ___ PLIP-Advisories mailing list plip-advisor...@lists.plone.org http://lists.plone.org/mailman/listinfo/plip-advisories
[PLIP-Advisories] Re: [Plone] #9249: Add TinyMCE as the default visual editor
#9249: Add TinyMCE as the default visual editor --+- Reporter: limi |Owner: robgietema Type: PLIP | Status: assigned Priority: minor |Milestone: 4.0 Component: Visual Editor (Kupu) | Resolution: Keywords:| --+- Comment(by xela7): A note - Before I installed the plip, I had an existing plone site. After intalling plip, the tiny mce was not in the control panel add on configuration or in the pull down list under the site configlet. Could be a problem for upgrades? -- Ticket URL: http://dev.plone.org/plone/ticket/9249#comment:20 Plone http://plone.org Plone Content Management System ___ PLIP-Advisories mailing list plip-advisor...@lists.plone.org http://lists.plone.org/mailman/listinfo/plip-advisories
[PLIP-Advisories] Re: [Plone] #9249: Add TinyMCE as the default visual editor
#9249: Add TinyMCE as the default visual editor --+- Reporter: limi |Owner: robgietema Type: PLIP | Status: assigned Priority: minor |Milestone: 4.0 Component: Visual Editor (Kupu) | Resolution: Keywords:| --+- Comment(by xela7): As noted above, I had an existing plone site. I deleted that site and added a new site. When trying to add a Page to the home page, (and also tried an even and got similar error) I received the following error: Traceback (innermost last): Module ZPublisher.Publish, line 127, in publish Module ZPublisher.mapply, line 77, in mapply Module ZPublisher.Publish, line 47, in call_object Module Products.CMFPlone.FactoryTool, line 446, in __call__ Module ZPublisher.mapply, line 77, in mapply Module ZPublisher.Publish, line 47, in call_object Module Products.CMFFormController.FSControllerPageTemplate, line 91, in __call__ Module Products.CMFFormController.BaseControllerPageTemplate, line 31, in _call Module Shared.DC.Scripts.Bindings, line 324, in __call__ Module Shared.DC.Scripts.Bindings, line 361, in _bindAndExec Module Products.CMFCore.FSPageTemplate, line 240, in _exec Module Products.CMFCore.FSPageTemplate, line 180, in pt_render Module Products.PageTemplates.PageTemplate, line 80, in pt_render Module zope.pagetemplate.pagetemplate, line 115, in pt_render - Warning: Macro expansion failed - Warning: type 'exceptions.KeyError': 'macro' Module zope.tal.talinterpreter, line 271, in __call__ Module zope.tal.talinterpreter, line 343, in interpret Module zope.tal.talinterpreter, line 888, in do_useMacro Module zope.tal.talinterpreter, line 343, in interpret Module zope.tal.talinterpreter, line 533, in do_optTag_tal Module zope.tal.talinterpreter, line 518, in do_optTag Module zope.tal.talinterpreter, line 513, in no_tag Module zope.tal.talinterpreter, line 343, in interpret Module zope.tal.talinterpreter, line 888, in do_useMacro Module zope.tal.talinterpreter, line 343, in interpret Module zope.tal.talinterpreter, line 533, in do_optTag_tal Module zope.tal.talinterpreter, line 518, in do_optTag Module zope.tal.talinterpreter, line 513, in no_tag Module zope.tal.talinterpreter, line 343, in interpret Module zope.tal.talinterpreter, line 954, in do_defineSlot Module zope.tal.talinterpreter, line 343, in interpret Module zope.tal.talinterpreter, line 533, in do_optTag_tal Module zope.tal.talinterpreter, line 518, in do_optTag Module zope.tal.talinterpreter, line 513, in no_tag Module zope.tal.talinterpreter, line 343, in interpret Module zope.tal.talinterpreter, line 858, in do_defineMacro Module zope.tal.talinterpreter, line 343, in interpret Module zope.tal.talinterpreter, line 954, in do_defineSlot Module zope.tal.talinterpreter, line 343, in interpret Module zope.tal.talinterpreter, line 533, in do_optTag_tal Module zope.tal.talinterpreter, line 518, in do_optTag Module zope.tal.talinterpreter, line 513, in no_tag Module zope.tal.talinterpreter, line 343, in interpret Module zope.tal.talinterpreter, line 946, in do_defineSlot Module zope.tal.talinterpreter, line 343, in interpret Module zope.tal.talinterpreter, line 533, in do_optTag_tal Module zope.tal.talinterpreter, line 518, in do_optTag Module zope.tal.talinterpreter, line 513, in no_tag Module zope.tal.talinterpreter, line 343, in interpret Module zope.tal.talinterpreter, line 858, in do_defineMacro Module zope.tal.talinterpreter, line 343, in interpret Module zope.tal.talinterpreter, line 533, in do_optTag_tal Module zope.tal.talinterpreter, line 518, in do_optTag Module zope.tal.talinterpreter, line 513, in no_tag Module zope.tal.talinterpreter, line 343, in interpret Module zope.tal.talinterpreter, line 888, in do_useMacro Module zope.tal.talinterpreter, line 343, in interpret Module zope.tal.talinterpreter, line 852, in do_condition Module zope.tal.talinterpreter, line 343, in interpret Module zope.tal.talinterpreter, line 954, in do_defineSlot Module zope.tal.talinterpreter, line 343, in interpret Module zope.tal.talinterpreter, line 533, in do_optTag_tal Module zope.tal.talinterpreter, line 518, in do_optTag Module zope.tal.talinterpreter, line 513, in no_tag Module zope.tal.talinterpreter, line 343, in interpret Module zope.tal.talinterpreter, line 852, in do_condition Module zope.tal.talinterpreter, line 343, in interpret Module zope.tal.talinterpreter, line 533, in do_optTag_tal Module zope.tal.talinterpreter, line 518, in do_optTag Module zope.tal.talinterpreter, line 513, in no_tag Module zope.tal.talinterpreter, line 343, in
[PLIP-Advisories] Re: [Plone] #9249: Add TinyMCE as the default visual editor
#9249: Add TinyMCE as the default visual editor --+- Reporter: limi |Owner: robgietema Type: PLIP | Status: assigned Priority: minor |Milestone: 4.0 Component: Visual Editor (Kupu) | Resolution: Keywords:| --+- Comment(by xela7): Note also, I switched to kupu in the site configlet and received the same error. I then switched back to tinymce and received the same error. -- Ticket URL: http://dev.plone.org/plone/ticket/9249#comment:22 Plone http://plone.org Plone Content Management System ___ PLIP-Advisories mailing list plip-advisor...@lists.plone.org http://lists.plone.org/mailman/listinfo/plip-advisories
[PLIP-Advisories] Re: [Plone] #9249: Add TinyMCE as the default visual editor
#9249: Add TinyMCE as the default visual editor --+- Reporter: limi |Owner: robgietema Type: PLIP | Status: assigned Priority: minor |Milestone: 4.0 Component: Visual Editor (Kupu) | Resolution: Keywords:| --+- Comment(by robgietema): This PLIP is ready for review. [http://svn.plone.org/svn/plone/buildouts/plone- coredev/branches/4.0/plips/plip9249-tinymce.txt] -- Ticket URL: http://dev.plone.org/plone/ticket/9249#comment:19 Plone http://plone.org Plone Content Management System ___ PLIP-Advisories mailing list plip-advisor...@lists.plone.org http://lists.plone.org/mailman/listinfo/plip-advisories
[PLIP-Advisories] Re: [Plone] #9249: Add TinyMCE as the default visual editor
#9249: Add TinyMCE as the default visual editor --+- Reporter: limi |Owner: robgietema Type: PLIP | Status: assigned Priority: minor |Milestone: 4.0 Component: Visual Editor (Kupu) | Resolution: Keywords:| --+- Changes (by grahamperrin): * cc: grahamper...@gmail.com (added) -- Ticket URL: http://dev.plone.org/plone/ticket/9249#comment:18 Plone http://plone.org Plone Content Management System ___ PLIP-Advisories mailing list plip-advisor...@lists.plone.org http://lists.plone.org/mailman/listinfo/plip-advisories
[PLIP-Advisories] Re: [Plone] #9249: Add TinyMCE as the default visual editor
#9249: Add TinyMCE as the default visual editor --+- Reporter: limi |Owner: robgietema Type: PLIP | Status: new Priority: minor |Milestone: 4.0 Component: Visual Editor (Kupu) | Resolution: Keywords:| --+- Changes (by esteele): * owner: = robgietema -- Ticket URL: http://dev.plone.org/plone/ticket/9249#comment:16 Plone http://plone.org Plone Content Management System ___ PLIP-Advisories mailing list plip-advisor...@lists.plone.org http://lists.plone.org/mailman/listinfo/plip-advisories
[PLIP-Advisories] Re: [Plone] #9249: Add TinyMCE as the default visual editor
#9249: Add TinyMCE as the default visual editor --+- Reporter: limi |Owner: Type: PLIP | Status: new Priority: minor |Milestone: 4.0 Component: Visual Editor (Kupu) | Resolution: Keywords:| --+- Comment(by esteele): Approved by FWT vote. -- Ticket URL: https://dev.plone.org/plone/ticket/9249#comment:14 Plone http://plone.org Plone Content Management System ___ PLIP-Advisories mailing list plip-advisor...@lists.plone.org http://lists.plone.org/mailman/listinfo/plip-advisories
