[ https://issues.apache.org/jira/browse/OFBIZ-3815?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12933128#action_12933128 ] Jacques Le Roux edited comment on OFBIZ-3815 at 11/17/10 3:15 PM: ------------------------------------------------------------------ Hi Sascha, I have commited your patches and elrte editor at r1036195 Here are some minor comments: * I have used jQuery 1.4.2 as I just reverted to this version * I have replaced PageTitleNavigate by PageTitleNavigateDataResources * We will need to localize labels and alert messages * I noticed + function destroyEditor() { + // TODO + } not sure it needs something or not * I have used 'maxi' as elrte Editor option to have table buttons by default To decodes HTML as input to elrte Editor, I have tried to use simple Freemarker replaces. I was surprised to see that some were not decoded. I then tried to use a regexp (because I though it was a multilines issue) but without more success... to be continued. was (Author: jacques.le.roux): Hi Sascha, I have commited your patches and elrte editor at r Here are some minor comments: * I have used jQuery 1.4.2 as I just reverted to this version * I have replaced PageTitleNavigate by PageTitleNavigateDataResources * We will need to localize labels and alert messages * I noticed + function destroyEditor() { + // TODO + } not sure it needs something or not * I have used 'maxi' as elrte Editor option to have table buttons by default To decodes HTML as input to elrte Editor, I have tried to use simple Freemarker replaces. I was surprised to see that some were not decoded. I then tried to use a regexp (because I though it was a multilines issue) but without more success... to be continued. > Content Navigation Tree implemented with jQuery > ----------------------------------------------- > > Key: OFBIZ-3815 > URL: https://issues.apache.org/jira/browse/OFBIZ-3815 > Project: OFBiz > Issue Type: Sub-task > Components: content > Affects Versions: jQuery > Reporter: Sascha Rodekamp > Assignee: Jacques Le Roux > Fix For: jQuery > > Attachments: OFBIZ-3815_ContentNavigationMigration_part1.patch, OFBIZ-3815_elrteEditorPlugin.zip, OFBIZ-3815_jqueryExample_ContentNavigation.patch, OFBIZ-3815_jqueryExample_ContentNavigation.patch, OFBIZ-3815_plugins.zip, OFBIZ-3815_plugins.zip, OFBIZ-3815_WebSiteCMSMigration_part2.patch, OFBIZ_3815 > > > Hi, > here is the first (test) patch for the jQuery implementation. > I changed the content navigation tree for a first test: > [https://localhost:8443/content/control/navigateContent] > The implementation was much easier than expected :-) > I choose the jsTree Plugin v. 1.0rc (in the next days their will be a final 1.0 release, that's why i choose the rc). > [http://www.jstree.com/] > I also added > {code} > jQuery.noConflict(); > {code} > to the libary. That overwrites the '$( ... )' operator to prevent conflicts with prototype. This line can be deleted after prototype is totally replaced. > For know we have to call jQuery via jQuery( .... ) instead of $( ... ). > To apply the patch the zip file have to copied to /images/jquery/ > it contains a plugin folder with the jsTree plugin > (should look like /images/jquery/plugins/jsTree) > So long > Sascha -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online. |
Free forum by Nabble | Edit this page |