Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [orion-dev] Client - pending changes are lost when switching files and auto-save is enabled

Very cool stuff, Tomer. Thanks for sharing those details!

John




From:        Tomer Ohana <tomero@xxxxxxx>
To:        Orion developer discussions <orion-dev@xxxxxxxxxxx>
Cc:        orion-dev-bounces@xxxxxxxxxxx
Date:        05/04/2015 11:31 AM
Subject:        Re: [orion-dev] Client - pending changes are lost when switching files and auto-save is enabled
Sent by:        orion-dev-bounces@xxxxxxxxxxx




I attached the image of the current design (orion.jpg).

The strategy we took in our project is not touching the Orion code itself, but always extending it from the outside.
This is crucial for upgrading Orion with the least amount of pain.

I can list some of the main issues we encountered from the top of my head:
1. Hard coded CSS styles in the code. The latest release improved that but there are still some remains which makes it harder to customize.
2. The entire application is not a SPA - switching between the code editor and settings for example causes a full page reload. We had to work around that.
3. Some JS objects (like the left navigator tree) seem customizable by accepting an options object, but there's no way from "outside" to change these options, they are all hard-coded.

This made us create a small overrides system - we basically use requirejs mapping and override Orion modules with our own, and patching/replacing existing object functions.

Our team would love to better collaborate with you, unfortunately we are very busy with our release.
After everything calms down we will try to cooperate more by providing feedback and submitting code.

Thanks! 


On Mon, May 4, 2015 at 5:29 PM, Anton McConville <Anton.McConville@xxxxxxxxxx> wrote:
Thanks Tomer.

It'd be great to see your design somehow, if you had a site for instance, or design doc that we could look at.


I'm sorry to hear that it hasn't been easy to customise - we're conscious of some aspects that are making that hard, and want to improve.


Your feedback would be invaluable for improving too, either in some notes, or by phone/skype. We have an open meeting once a week that you're welcome to dial into.


If you have time, please let us know what has been difficult, and how you think things could be improved.


Anton



Inactive hide details for Tomer Ohana ---2015/05/04 09:58:37 AM---I will try to submit changes to Gerrit, thanks! Anton, to ansTomer Ohana ---2015/05/04 09:58:37 AM---I will try to submit changes to Gerrit, thanks! Anton, to answer our question, we are currently developing a product that invol


From:

Tomer Ohana <
tomero@xxxxxxx>

To:

Orion developer discussions <
orion-dev@xxxxxxxxxxx>

Date:

2015/05/04 09:58 AM

Subject:

Re: [orion-dev] Client - pending changes are lost when switching files and auto-save is enabled






I will try to submit changes to Gerrit, thanks!

Anton, to answer our question, we are currently developing a product that involves an online IDE and we chose Orion.
We customize and extend the client heavily (unfortunately it's not that easy).
The server we wrote from scratch since it needs to work with our own backend.
I don't know if it's allowed to send images, but if so let me know and I can show our design.


On Mon, May 4, 2015 at 4:13 PM, Adrian Aichner <
adrian.aichner@xxxxxxxxx> wrote:
This might also be interesting in this context:

[Subject: auto-save-include-big-deletions](
https://lists.gnu.org/archive/html/emacs-devel/2009-07/msg01091.html)



On 04.05.2015 14:40, John Arthorne wrote:

Hi Tomer, 

Yikes, that sounds serious. Yes please enter a bugzilla bug report for it. You can propose fixes either with a pull request, or even better by pushing to our Gerrit for review:
 

https://wiki.eclipse.org/Orion/Contributing_Code#Submit_your_changes_to_Gerrit 

Thanks!
 

John
 



From:        
Tomer Ohana <tomero@xxxxxxx> 
To:        
orion-dev@xxxxxxxxxxx 
Date:        
05/03/2015 09:20 AM 
Subject:        
[orion-dev] Client - pending changes are lost when switching files and auto-save is enabled 
Sent by:        
orion-dev-bounces@xxxxxxxxxxx 




Hi everyone,

We're using Orion in one of our products and found a serious bug. I think it originated from this commit: 

Editor is not dirty after failed save attempty 

In short, when user edits a file, with auto-save enabled, the editor is marked clean although there are still unsaved changes.
This happens when the save function is called between user edits, meaning the user still edits the file while there is an outgoing save request.
When the response returns, the editor is marked clean (
inputManager:357) although there are still unsaved changes.
If the user switches to a different file or closes the browser window, those unsaved changes are lost.
Although it seems tricky to reproduce, it happens to us all the time.

I'm new here so I just wanted to ask if anyone encountered it too, or should I open a bug in BugZilla? Is it possible to suggest a fix via a pull request?

Thanks for the help,
Tomer.
_______________________________________________
orion-dev mailing list

orion-dev@xxxxxxxxxxx
To change your delivery options, retrieve your password, or unsubscribe from this list, visit

https://dev.eclipse.org/mailman/listinfo/orion-dev 


_______________________________________________
orion-dev mailing list

orion-dev@xxxxxxxxxxx
To change your delivery options, retrieve your password, or unsubscribe from this list, visit

https://dev.eclipse.org/mailman/listinfo/orion-dev


_______________________________________________
orion-dev mailing list

orion-dev@xxxxxxxxxxx
To change your delivery options, retrieve your password, or unsubscribe from this list, visit

https://dev.eclipse.org/mailman/listinfo/orion-dev



--
Head of front end development,
Wix Code

_______________________________________________
orion-dev mailing list

orion-dev@xxxxxxxxxxx
To change your delivery options, retrieve your password, or unsubscribe from this list, visit

https://dev.eclipse.org/mailman/listinfo/orion-dev


_______________________________________________
orion-dev mailing list

orion-dev@xxxxxxxxxxx
To change your delivery options, retrieve your password, or unsubscribe from this list, visit

https://dev.eclipse.org/mailman/listinfo/orion-dev



--
Head of front end development,
Wix Code

[attachment "orion.jpg" deleted by John Arthorne/Ottawa/IBM] _______________________________________________
orion-dev mailing list
orion-dev@xxxxxxxxxxx
To change your delivery options, retrieve your password, or unsubscribe from this list, visit
https://dev.eclipse.org/mailman/listinfo/orion-dev

Back to the top