[ 
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.

Reply via email to