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

Bug 382302

Summary: Tooltip help appears too fast
Product: [ECD] Orion Reporter: Ken Walker <ken_walker>
Component: ClientAssignee: Ken Walker <ken_walker>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: simon_kaegi, susan
Version: 0.5Flags: simon_kaegi: review+
Target Milestone: 0.5 RC3   
Hardware: All   
OS: All   
Whiteboard:
Attachments:
Description Flags
options tooltip over user menu none

Description Ken Walker CLA 2012-06-11 16:43:45 EDT
Created attachment 217186 [details]
options tooltip over user menu

Typically with a tooltip or hover help, you pause the pointing device around the hotspot and wait a bit for the tooltip.  If you have been using a product for a while, you shouldn't be provided immediately with a tooltip and I find that the timeout currently on Orion to be just that. Too fast.

The most annoying one I find is going to my name to get to Settings.  As soon as I get there it say "options" which pops up over my navigation to "settings".

If the pause was more like the second my other OS tooltips seem to be it would likely not pop up at all.
Comment 1 Susan McCourt CLA 2012-06-14 17:25:49 EDT
I looked into this.  Fortunately everything goes through CommandTooltip.  So you just want to change the showDelay property.  I thought it'd be a one liner, but we also have to make sure our polling timer triggers after the initial delay, hence two line change.

here's a patch based on a different branch, so the line numbers might be slightly different, but you'll get the gist.  Tweak that first showDelay number to your heart's content.  The second change should always be showDelay + 500.

diff --git a/bundles/org.eclipse.orion.client.core/web/orion/commands.js b/bundles/org.eclipse.orion.client.core/web/orion/commands.js
index 44b3659..a1f78e7 100644
--- a/bundles/org.eclipse.orion.client.core/web/orion/commands.js
+++ b/bundles/org.eclipse.orion.client.core/web/orion/commands.js
@@ -146,6 +146,7 @@
 		constructor : function() {
 			this.inherited(arguments);
 			this.options = arguments[0] || {};
+			this.showDelay = 1500;
 		},
 		
 		_onUnHover: function(evt){
@@ -190,7 +191,7 @@
 					this.polling = false;
 					this.close();
 				}
-			}), 1000);
+			}), this.showDelay + 500);
 		},
 		
 		_stillInDocument: function(node) {
Comment 2 Ken Walker CLA 2012-06-14 20:25:19 EDT
Playing around with it, the standard 1000 seems to work pretty well. 1500 seems a bit long.
Comment 3 Susan McCourt CLA 2012-06-15 11:10:59 EDT
(In reply to comment #2)
> Playing around with it, the standard 1000 seems to work pretty well. 1500 seems
> a bit long.

yeah, I put in a big number just so I could tell it was working.
Comment 4 Ken Walker CLA 2012-06-18 13:03:31 EDT
See http://git.eclipse.org/c/orion/org.eclipse.orion.client.git/commit/?h=Bug382302

Or cherry pick 	da0a25111bfdcf9030455ba4c6226f2b4f2917b5 to see
Comment 5 Ken Walker CLA 2012-06-18 13:41:47 EDT
Simon said "release the code"
Comment 6 Ken Walker CLA 2012-06-18 13:43:46 EDT
Pushed to master