Fix Bug 368869 - Page-specific command link bar is inaccessible

2 files changed, 6 insertions, 6 deletions

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 2946357..520d89b 100644
--- a/bundles/org.eclipse.orion.client.core/web/orion/commands.js
+++ b/bundles/org.eclipse.orion.client.core/web/orion/commands.js
@@ -903,7 +903,7 @@ define(['require', 'dojo', 'dijit', 'orion/util', 'dijit/Menu', 'dijit/form/Drop
 		},
 		_addTool: function(parent, forceText, name, context, activeCommandClass, inactiveCommandClass) {
 			context.handler = context.handler || this;
-			var link = dojo.create("a");
+			var link = dojo.create("a", {tabindex: "0", role: "button", href: "javascript:void(0)"}); // the link must have an href for keyboard use
 			link.id = this.name+"link";
 			var image = null;
 			if (forceText || !this.hasImage()) {
@@ -985,7 +985,7 @@ define(['require', 'dojo', 'dijit', 'orion/util', 'dijit/Menu', 'dijit/form/Drop
 			context.handler = context.handler || this;
 			var element;
 			if (this.hrefCallback) {
-				element = dojo.create("a");
+				element = dojo.create("a", {tabindex: "0", role: "button", href: "javascript:void(0)"}); // the link must have an href for keyboard use
 				dojo.addClass(element, "commandLink");
 			} else {
 				element = dojo.create("button");
diff --git a/bundles/org.eclipse.orion.client.core/web/plugin/list.js b/bundles/org.eclipse.orion.client.core/web/plugin/list.js
index 1c04b5d..437f03b 100644
--- a/bundles/org.eclipse.orion.client.core/web/plugin/list.js
+++ b/bundles/org.eclipse.orion.client.core/web/plugin/list.js
@@ -47,13 +47,13 @@ dojo.addOnLoad(function() {
 		if (pageActions) {
 			dojo.place('<input type="text" id="installUrlTextBox" value="Type a plugin URL here" style="width:16em;"></input>',
 				pageActions, "last");
-			dojo.place('<a id="installButton" title="Install the plugin specified in the edit box">Install</a>',
+			dojo.place('<a id="installButton" title="Install the plugin specified in the edit box" tabindex="0" role="button" href="javascript:void(0)">Install</a>',
 				pageActions, "last");
-			dojo.place('<a id="uninstallButton" title="Uninstall the selected plugins">Uninstall</a>',
+			dojo.place('<a id="uninstallButton" title="Uninstall the selected plugins" tabindex="0" role="button" href="javascript:void(0)">Uninstall</a>',
 				pageActions, "last");
-			dojo.place('<a id="reloadButton" title="Reload the selected plugin from its URL">Reload</a>',
+			dojo.place('<a id="reloadButton" title="Reload the selected plugin from its URL" tabindex="0" role="button" href="javascript:void(0)">Reload</a>',
 				pageActions, "last");
-			dojo.place('<a id="copyButton" title="Copy the selected location into the install edit box">Copy Location</a>',
+			dojo.place('<a id="copyButton" title="Copy the selected location into the install edit box" tabindex="0" role="button" href="javascript:void(0)">Copy Location</a>',
 				pageActions, "last");