Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 344306 - http://www.eclipse.org/orion editor undo/redo buttons don't work
Summary: http://www.eclipse.org/orion editor undo/redo buttons don't work
Status: RESOLVED WONTFIX
Alias: None
Product: Orion
Classification: ECD
Component: Doc (show other bugs)
Version: 0.2   Edit
Hardware: PC Windows 7
: P3 normal (vote)
Target Milestone: 0.5   Edit
Assignee: Ken Walker CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-04-29 12:00 EDT by Dean Roberts CLA
Modified: 2012-05-11 09:36 EDT (History)
3 users (show)

See Also:


Attachments
mockup of landing page with embedded editor (109.14 KB, image/png)
2011-05-05 11:57 EDT, Susan McCourt CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Dean Roberts CLA 2011-04-29 12:00:25 EDT
Build Identifier: 

If I navigate to:

http://www.eclipse.org/orion/

Using Firefox 3.6.16 on Windows 7 the undo/redo buttons do not work.
John Arthorne also confirms it does not work for him using Windows 7, Chrome 11

Setting severity to critical because it is darned embarrassing.  Feel free to downgrade :-) 


Reproducible: Always
Comment 1 Susan McCourt CLA 2011-05-02 12:18:41 EDT
Nathan, have these buttons ever worked?
Note that Ctrl-Z and Ctrl-Y don't work either.  

I think the answer to this is to embed the editorContainer now that bug 337647 is completed.  Nathan, there is an example, http://localhost:8080/examples/minimaleditor.html, that shows how this works.  I plan to update this example to use prettier visuals, and I would hope you could base the embedded landing page editor on this.
Comment 2 Nathan Gervais CLA 2011-05-02 13:00:17 EDT
Yes those buttons used to work, thanks for reporting this.

Totally agree, having a minimal editor is a good thing, I should migrate Orion to its code.
Comment 3 Susan McCourt CLA 2011-05-05 11:50:11 EDT
(In reply to comment #2)
> Yes those buttons used to work, thanks for reporting this.
> 
> Totally agree, having a minimal editor is a good thing, I should migrate Orion
> to its code.

Nathan, see bug 340926.  I now have an html page that embeds an editor with js, java, and css syntax highlighting, and keyword content assist for css and js.  Undo and redo are working in this example (though I have not placed buttons in the toolbar for it).

The example puts a "save" link in the top editor bar, but you could just as easily put undo, redo, or whatever else you want there.  I think this should be the basis for embedding the editor and can work with you on any questions or problems you encounter.
Comment 4 Susan McCourt CLA 2011-05-05 11:57:22 EDT
Created attachment 194848 [details]
mockup of landing page with embedded editor

While working on the "skinny header" for the embedded editor case, our designer did a mockup of the landing page to show what it looks like in context.

She also suggested using a dark border around the embedded editor to fit with the header visual, you can do what you want with this idea....
Comment 5 Susan McCourt CLA 2011-05-05 12:02:27 EDT
Nathan, if you want to show off HTML syntax highlighting, you can use the plugin

- org.eclipse.orion.client.core/static/plugins/htmlSyntaxHighlightPlugin.html
Comment 6 Nathan Gervais CLA 2011-05-11 11:39:16 EDT
I attempted to update the editor today loosely following the instructions at http://wiki.eclipse.org/Orion/How_Tos/Setup_Orion_Client_Hosted_Site_on_OrionHub with little success.

For a minimal editor I found the requirement in order to get this working to be quite the opposite.  Having to include 
/org.eclipse.orion.client.core/static 
/org.eclipse.orion.client.editor/web
as well as dojo locally does not seem minimal to me.

In the mean time i've removed the undo buttons.
Comment 7 Susan McCourt CLA 2011-05-11 13:21:15 EDT
(In reply to comment #6)
> I attempted to update the editor today loosely following the instructions at
> http://wiki.eclipse.org/Orion/How_Tos/Setup_Orion_Client_Hosted_Site_on_OrionHub
> with little success.
> 
> For a minimal editor I found the requirement in order to get this working to be
> quite the opposite.  Having to include 
> /org.eclipse.orion.client.core/static 
> /org.eclipse.orion.client.editor/web
> as well as dojo locally does not seem minimal to me.

That how-to is for folks who want to run any part of the orion client in an environment hosted on orion hub.

For the editor case, you don't need all of those files, but you do need...
<script src="/editor/samples/styler.js"></script>
<script src="/editor/samples/rulers.js"></script>
<script src="/editor/samples/undoStack.js"></script>
<script src="/editor/js/model.js"></script>
<script src="/editor/js/editor.js"></script>	
<script src="/js/editorFeatures.js"></script>
<script src="/js/contentAssist.js"></script>
<script src="/js/contentAssist/coreContentAssist.js"></script>
<script src="/js/editorContainer.js"></script>
<script src="/examples/embeddededitor.js"></script>

The big issue here is dojo.  I opened bug 345472 because I think our references to dojo in the latter 5 files is fairly random and we could make an effort to stop using dojo convenience methods in these files.
Comment 8 Simon Kaegi CLA 2011-06-15 11:09:39 EDT
Sorry Dean -- downgrading ;)
Not that's it not embarrassing but it's not preventing a release (and keeps on freaking me out when I triage).

Could you take a look again to see if the minimal editor is now a sufficiently good example to get this working.
Comment 9 Simon Kaegi CLA 2011-10-20 14:14:33 EDT
We need to update the editor here.
Comment 10 Ken Walker CLA 2012-05-11 09:36:51 EDT
So this is moot now that we've currently removed the editor from the page.