Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 350636

Summary: [client] Remove dojo dependency from TextMateStyler and HtmlGrammar
Product: [ECD] Orion Reporter: Mihai Sucan <mihai.sucan>
Component: EditorAssignee: Mark Macdonald <mamacdon>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: eclipse.felipe, mamacdon, mihai.sucan
Version: unspecified   
Target Milestone: 0.3 M1   
Hardware: All   
OS: All   
URL: https://github.com/eclipse/orion.client/pull/6
Whiteboard:

Description Mihai Sucan CLA 2011-06-28 16:29:53 EDT 
Build Identifier: 

For Mozilla integration we use Orion TextView, without dojo, and no orion.editor code.

We would like the TextMateStyler dojo dependency removed - it's something really minor. The cost of integrating dojo into our codebase is much higher than fixing this here in Orion's code.

This is needed so we can enable HTML syntax highlighting in the Orion integration code (with the TextView component).

Will submit a pull request.

Thank you!

Reproducible: Always
Comment 1 Mihai Sucan CLA 2011-06-28 16:36:28 EDT 
Pull request submitted:

https://github.com/eclipse/orion.client/pull/6

I have also updated the TextView demo to include an HTML file, so users can play with this. TextMateStyler and HtmlGrammar no longer depend on the editor itself, which really fits with the idea of a TextView + syntax highlighters + and higher-level editor features.

Please let me know if further changes are needed for this to land. With this code we can use HTML syntax highlighting in our Orion integration work - which makes the patch really important for us. Thank you very much!
Comment 2 Mark Macdonald CLA 2011-06-29 10:13:02 EDT 
The patch looks good. Thanks Mihai!

But I need you to add a comment here confirming the usual legal song and dance, something like:

> "I wrote all this code and have the rights to contribute it to Eclipse under the eclipse.org web site terms of use."
Comment 3 Mihai Sucan CLA 2011-06-29 10:59:49 EDT 
I wrote all this code and have the rights to contribute it to Eclipse under the eclipse.org web site terms of use.

Thanks for looking into the patch! Looking forward to see this land.