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

Bug 490492

Summary: Orion should show it's version
Product: [ECD] Orion Reporter: Mike Wilson <Mike_Wilson>
Component: ClientAssignee: Curtis Windatt <curtis.windatt.public>
Status: RESOLVED FIXED QA Contact:
Severity: enhancement    
Priority: P2 CC: ahunter.eclipse, curtis.windatt.public, Silenio_Quarti, steve_northover
Version: unspecified   
Target Milestone: 13.0   
Hardware: Other   
OS: All   
Whiteboard:
Attachments:
Description Flags
Fix
none
Putting it in the settings status pane
none
Patch to display in status pane wrapped in darklaunch flag none

Description Mike Wilson CLA 2016-03-26 21:49:39 EDT
I wanted to check what version of Orion I was running on a microcomputer in my house. I expected to find an "About Orion..." or similar menu entry, but did not.

This seems like it would be useful. Particularly, if it indicated any other relevant information about the Orion (e.g. file size constraints, whether wildcard hosting was supported, etc.)
Comment 1 Steve Northover CLA 2016-03-28 11:44:02 EDT
Seems reasonable.
Comment 2 Anthony Hunter CLA 2016-03-28 14:12:46 EDT
We already have the hotkey CTRL+SHIFT+F1 to bring up the about page. We just need a new "About Orion..." menu entry.
Comment 3 Steve Northover CLA 2016-03-28 14:17:45 EDT
If we add a menu item that invokes this, it should be something like 'System information'.  People expect 'About XXX' to be a small dialog box.
Comment 4 Curtis Windatt CLA 2016-03-31 11:06:25 EDT
Created attachment 260647 [details]
Fix

This patch adds the menu item.  However, on WebIDE the user menu is not available and the orion node server (running on orion.eclipse.org) does not make the about page available.  In the host it times out and when self hosting the page cannot be found.
Comment 5 Steve Northover CLA 2016-04-06 12:47:19 EDT
SSQ?  Shouldn't this just work?
Comment 6 Silenio Quarti CLA 2016-04-06 12:52:33 EDT
We need to add a couple of endpoints in the node server (/about/about.html and /version).

And for the WebIDE, we need to find a place to add the menu item entry.
Comment 7 Steve Northover CLA 2016-10-07 13:11:49 EDT
How about File->About?
Comment 8 Steve Northover CLA 2016-10-07 13:33:59 EDT
Tools->Version or how about on the settings page in the title?  Just need it somewhere.
Comment 9 Curtis Windatt CLA 2016-10-13 16:25:43 EDT
Created attachment 264835 [details]
Putting it in the settings status pane

What about the status pane?  Only on the settings page.
Comment 10 Curtis Windatt CLA 2016-10-13 16:29:42 EDT
Created attachment 264836 [details]
Patch to display in status pane wrapped in darklaunch flag

If you like the way this looks I'll put it into master with the darklaunch flag.

The only problem is that when self hosting the version route will 404.  This is going to be a problem no matter what solution we take though.
Comment 11 Steve Northover CLA 2016-10-13 16:47:32 EDT
Setting is fine.  Let's make it possible to see the version without having to know an end point.  Please just fix this very old and obvious problem.

We can tweak where it appears later.
Comment 12 Curtis Windatt CLA 2016-10-13 17:14:02 EDT
http://git.eclipse.org/c/orion/org.eclipse.orion.client.git/commit/?id=27c24d247aa87eec4b4d1ac2755e6b25cde86b5e

You must set localStorage.darklaunch=true.

We cannot access the version endpoint when self hosting and the xhr 404 will show up in the console.
Comment 13 Steve Northover CLA 2016-10-14 12:48:44 EDT
Seems fine.  SSQ?
Comment 14 Silenio Quarti CLA 2016-10-14 15:31:07 EDT
(In reply to Steve Northover from comment #13)
> Seems fine.  SSQ?

Sure. go ahead.  Note that the version is gone as soon as a command changes the status (i.e. Reload All in plugins page)
Comment 15 Steve Northover CLA 2016-10-14 16:51:59 EDT
That's a bug.  Are there other places that assume the empty string should be restored?
Comment 16 Steve Northover CLA 2016-10-17 09:32:20 EDT
Can we release this and address the bug later (or now if the change is easy)?
Comment 17 Curtis Windatt CLA 2016-10-17 13:50:50 EDT
http://git.eclipse.org/c/orion/org.eclipse.orion.client.git/commit/?id=706a98effaa94545588fed3c0ecab089a0287753
Fixed in master

Removed the dark launch flag
If the user switches categories the version string will be added back.  So in Silenio's example, you see the verison, run reload all plugins, version is gone, click on any category, version string is back.  I didn't see a way to have a default status message and there are very few commands on the settings page that affect it.