Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [orion-dev] Issue with embedding Orion.Client

I fixed https://bugs.eclipse.org/bugs/show_bug.cgi?id=383705. The default-theme.css file belongs in the editor bundle.

I will investigate whether the editor can have a default size to avoid it showing up blank when the parent has no size.

Something else we should investigate is whether other pieces of the orion editor that currently live in the core bundle can be factored out and moved to the editor bundle.

Silenio



From:        John Arthorne/Ottawa/IBM@IBMCA
To:        Orion developer discussions <orion-dev@xxxxxxxxxxx>,
Date:        06/28/2012 03:58 PM
Subject:        Re: [orion-dev] Issue with embedding Orion.Client
Sent by:        orion-dev-bounces@xxxxxxxxxxx




The problems seem to be wider than just the editor. We have examples that show embedding a very basic Orion editor and I think they are fairly straight-forward. The trick is when you want to start pulling in more scripts for outlining, highlighting, content assist, etc. In theory require.js helps with wiring up all the inter-script dependencies, but you need a main require.js configuration that maps all the source locations to module names. The scarier problem that Andy mentioned is that even once you do this, there are timing problems if the scripts aren't loaded/run in the same order as we have in Orion. These are the cases we need to track down and figure out how to make our code less brittle.



Mike Wilson/Ottawa/IBM@IBMCA
Sent by: orion-dev-bounces@xxxxxxxxxxx

06/28/2012 10:20 AM

Please respond to
Orion developer discussions <orion-dev@xxxxxxxxxxx>

To
Orion developer discussions <orion-dev@xxxxxxxxxxx>
cc
Subject
Re: [orion-dev] Issue with embedding Orion.Client







Consumption of the editor in multiple contexts is very important to us. Silenio and/or Simon, what can we do so that makes this a) easier to consume and b) documented well enough that people can actually make progress.

McQ.


Inactive hide details for John J Barton ---06/27/2012 08:28:23 PM---On Wed, Jun 27, 2012 at 5:09 PM, Andy Clement <andrew.clemeJohn J Barton ---06/27/2012 08:28:23 PM---On Wed, Jun 27, 2012 at 5:09 PM, Andy Clement <andrew.clement@xxxxxxxxx>wrote: > When I was pulling

From:
John J Barton <johnjbarton@xxxxxxxxxxxxxxx>
To:
Orion developer discussions <orion-dev@xxxxxxxxxxx>,
Date:
06/27/2012 08:28 PM
Subject:
Re: [orion-dev] Issue with embedding Orion.Client
Sent by:
orion-dev-bounces@xxxxxxxxxxx






On Wed, Jun 27, 2012 at 5:09 PM, Andy Clement <
andrew.clement@xxxxxxxxx> wrote:
When I was pulling the editor out of Orion for use in another project,
I hit these issues.  I do have a minimal project that is the pulled
out editor that I can probably share with you, if you'd find it useful
- but basically I pulled and re-arranged as necessary to give myself
the minimal set of files I needed.

Once pulled out though there are many more steps and gotchas as you
attempt to integrate the other pieces that makes it a more
sophisticated editing experience: syntax highlighting,
rulers/annotations (and their highlighting in the editor), jslint,
content assist, etc...  Right now you have to do many things in the
*exact* same order that orion does otherwise your event listeners
register in an incompatible order. I plan to raise some bugzillas on
this when I get some time.

> I can't figure out default-theme.css however. The file lives at:
> ./orion.client/bundles/org.eclipse.orion.client.core/web/css/default-theme.css
> but it is called in by
> ./orion.client/bundles/org.eclipse.orion.client.editor/web/orion/editor/editor.css:@import
> "../../css/default-theme.css";
> So I can't see how this can ever work.

I think this only works when the server is running and the different
bundles are 'merged' and being hosted from the same place (then the
relative path is correct in server terms).


Ah, thanks:

https://bugs.eclipse.org/bugs/show_bug.cgi?id=383705

BTW we are pretty successful using chrome-extension://<id> as our 'server', once we sort out the path stuff.

jjb
 


Andy


> On Wed, Jun 27, 2012 at 2:10 PM, Zachariah Moreno <
xachmoreno@xxxxxxxxx>
> wrote:
>>
>> Thank You Silenio,
>>
>> for putting together that patch, which I just applied. I am still
>> receiving a failed to load resource errors for the textview & editor .js
>> files which you can see in the attached screenshot. I'm not having any
>> problems with embedding the editor into a standard webpage (on Orion local
>> or remote), but instead from within our extension. None-the-less, this just
>> seems to be a relative path issue, that I am trying to fix through trial &
>> error for now.
>>
>> Thanks again,
>>
>> --
>> Zachariah Moreno
>> Sent with Sparrow
>>
>> On Wednesday, June 27, 2012 at 1:08 PM, Silenio Quarti wrote:
>>
>> Hi Zach,
>>
>> I believe this is not a problem with the version of require.js. The
>> problem is with the way the files are layed out in the file system (or
>> server).
>>
>> How are you mounting the files on your server?  I was able to open
>> panel.html by making the changes in the attached patch and creating an orion
>> site with the following mappings:
>>
>>
>> /Orion-X    ->   /
>> /Orion-X/orion.client/bundles/org.eclipse.orion.client.editor/web  ->  /
>> /Orion-X/orion.client/bundles/org.eclipse.orion.client.core/web  -> /
>>
>> Silenio
>>
>>
>>
>>
>>
>>
>> From:        John J Barton <
johnjbarton@xxxxxxxxxxxxxxx>
>> To:        Orion developer discussions <
orion-dev@xxxxxxxxxxx>,
>> Date:        06/26/2012 06:15 PM
>> Subject:        Re: [orion-dev] Issue with embedding Orion.Client
>> Sent by:        
orion-dev-bounces@xxxxxxxxxxx
>> ________________________________
>>
>>
>>
>>
>>
>> On Tue, Jun 26, 2012 at 3:11 PM, Zachariah Moreno <
xachmoreno@xxxxxxxxx>
>> wrote:
>> Thank You John,
>>
>> I hadn't checked my version of Require. I just updated my copy to the
>> latest (v2.0.2) which is still presenting the same error. It was my
>> understanding that orion.client is dependent on require.js, but it is not
>> included in the repo, is this correct?
>>
>> Try the one in
>>
>> /orion.client/bundles/org.eclipse.orion.client.core/web/requirejs/require.js
>>
>>
>> The paths you mention are not giving me a Failed to Load Resource, is that
>> what you're getting?
>>
>> No, I just get undefined arguments, no other errors.
>>
>> jjb
>>
>>
>> Zach
>>
>> --
>> Zachariah Moreno
>> Sent with Sparrow
>>
>> On Tuesday, June 26, 2012 at 2:41 PM, John J Barton wrote:
>>
>>
>>
>> On Tue, Jun 26, 2012 at 1:11 PM, Zachariah Moreno <
xachmoreno@xxxxxxxxx>
>> wrote:
>> Hello All,
>>
>> I have been attempting to embed orion.client into a Chrome DevTools panel,
>> as part of my efforts for Google Summer of Code. The first method I used was
>> outlined by Felipe Heidrich in his blog post Using the Orion Editor with
>> Almond, which was a success in most ways but not others. So I switched my
>> method of embedding to use the the example presented by John Arthorne (thank
>> you btw) in his more recent post Examples of Embedding The Orion Editor,
>> which is more fitting for use in conjunction with the API exposed in
>> DevTools. Last week, John Barton was kind enough to spend some time with me
>> on this project. We determined John Arthorne's Embedded editor example was
>> the most fitting for our desired outcome & the switch was made successfully
>> in this repo. However, the file structure of orion.client was not maintained
>> to get this working in a short period of time. After reviewing this
>> progress, John Barton suggested I go through the process again, while
>> maintaining the orion.client file structure to allow for ease of updates in
>> the future (which I completely agree with). I have created a separate repo
>> for this change & updated all of the paths to reflect this, but I am getting
>> Uncaught TypeError: Cannot read property 'CssContentAssistProvider' of
>> undefined
>> chrome-extension://hnnbddcdmjkbcckijnphgophmegpflhe/embeddededitor.js:47.
>> From what I can tell, `mCSSContentAssist.CssContentAssistProvider()` is only
>> referenced in embeddededitor.js & the error never came up when I did my
>> first implementation. If anyone has any thoughts or advice please don't
>> hesitate to share.
>>
>> embeddededitor.js uses require.js with module ids like:
>>
>> "orion.client/bundles/org.eclipse.orion.client.editor/web/orion/textview/textView"
>> All of the arguments that match these ids are 'undefined'.
>>
>> I have no idea how this can happen (require.js is loading the files but
>> not passing the exports to the arguments of define).  I suggest checking the
>> version of require.js against the one used by Orion.
>>
>> jjb
>>
>>
>> Thank You Kindly,
>>
>> --
>> Zachariah Moreno
>> Sent with Sparrow
>>
>>
>> _______________________________________________
>> orion-dev mailing list
>>
orion-dev@xxxxxxxxxxx
>>
https://dev.eclipse.org/mailman/listinfo/orion-dev
>>
>>
>> _______________________________________________
>> orion-dev mailing list
>>
orion-dev@xxxxxxxxxxx
>>
https://dev.eclipse.org/mailman/listinfo/orion-dev
>>
>>
>> _______________________________________________
>> orion-dev mailing list
>>
orion-dev@xxxxxxxxxxx
>>
https://dev.eclipse.org/mailman/listinfo/orion-dev
>>
>> _______________________________________________
>> orion-dev mailing list
>>
orion-dev@xxxxxxxxxxx
>>
https://dev.eclipse.org/mailman/listinfo/orion-dev
>>
>> _______________________________________________
>> orion-dev mailing list
>>
orion-dev@xxxxxxxxxxx
>>
https://dev.eclipse.org/mailman/listinfo/orion-dev
>>
>> Attachments:
>> - patch_orionx.txt
>>
>>
>>
>> _______________________________________________
>> orion-dev mailing list
>>
orion-dev@xxxxxxxxxxx
>>
https://dev.eclipse.org/mailman/listinfo/orion-dev
>>
>
>
> _______________________________________________
> orion-dev mailing list
>
orion-dev@xxxxxxxxxxx
>
https://dev.eclipse.org/mailman/listinfo/orion-dev
>
_______________________________________________
orion-dev mailing list

orion-dev@xxxxxxxxxxx
https://dev.eclipse.org/mailman/listinfo/orion-dev
_______________________________________________
orion-dev mailing list
orion-dev@xxxxxxxxxxx

https://dev.eclipse.org/mailman/listinfo/orion-dev
_______________________________________________
orion-dev mailing list
orion-dev@xxxxxxxxxxx

https://dev.eclipse.org/mailman/listinfo/orion-dev
_______________________________________________
orion-dev mailing list
orion-dev@xxxxxxxxxxx
https://dev.eclipse.org/mailman/listinfo/orion-dev


Back to the top