From e30cd65994180284a778aea50db3ff0877b13c12 Mon Sep 17 00:00:00 2001
From: jtrobinson <jtrobinson629@gmail.com>
Date: Thu, 16 May 2013 09:19:52 -0700
Subject: [PATCH] Focus returns to Shortcut Help button after Glossary window
 closes. when the Glossary OPENS, focus moves to the titleBarOverlay as with
 all other module windows.

---
 .../main/events/ShortcutEvent.as              |  2 ++
 .../main/views/MainApplicationShell.mxml      |  2 +-
 .../bigbluebutton/main/views/MainToolbar.mxml |  5 +++++
 .../main/views/ShortcutHelpWindow.mxml        | 19 +++++++++++++++++++
 4 files changed, 27 insertions(+), 1 deletion(-)

diff --git a/bigbluebutton-client/src/org/bigbluebutton/main/events/ShortcutEvent.as b/bigbluebutton-client/src/org/bigbluebutton/main/events/ShortcutEvent.as
index 1ca33d13e4..867225d986 100755
--- a/bigbluebutton-client/src/org/bigbluebutton/main/events/ShortcutEvent.as
+++ b/bigbluebutton-client/src/org/bigbluebutton/main/events/ShortcutEvent.as
@@ -71,6 +71,8 @@ package org.bigbluebutton.main.events {
 		public static const CHANGE_FONT_COLOUR:String = 'CHANGE_FONT_COLOUR';
 		public static const SEND_MESSAGE:String = 'SEND_MESSAGE';
 		
+		public static const FOCUS_SHORTCUT_BUTTON:String = 'FOCUS_SHORTCUT_BUTTON';
+		
 		// Temporary string to help fix chat message navigation for screen readers
 		public static const CHAT_DEBUG:String = 'CHAT_DEBUG';
 		
diff --git a/bigbluebutton-client/src/org/bigbluebutton/main/views/MainApplicationShell.mxml b/bigbluebutton-client/src/org/bigbluebutton/main/views/MainApplicationShell.mxml
index 77dfbaa1ef..5dbec787a6 100755
--- a/bigbluebutton-client/src/org/bigbluebutton/main/views/MainApplicationShell.mxml
+++ b/bigbluebutton-client/src/org/bigbluebutton/main/views/MainApplicationShell.mxml
@@ -222,7 +222,7 @@ with BigBlueButton; if not, see <http://www.gnu.org/licenses/>.
 				mdiCanvas.windowManager.add(scWindow);
 				mdiCanvas.windowManager.absPos(scWindow, mdiCanvas.width/2 - 150, mdiCanvas.height/2 - 150);
 				
-				scWindow.focusCategories();
+				scWindow.focusHead();
 			}
 			
 			private function toggleFullScreen():void{
diff --git a/bigbluebutton-client/src/org/bigbluebutton/main/views/MainToolbar.mxml b/bigbluebutton-client/src/org/bigbluebutton/main/views/MainToolbar.mxml
index cb4056ef38..3e37ca45cb 100755
--- a/bigbluebutton-client/src/org/bigbluebutton/main/views/MainToolbar.mxml
+++ b/bigbluebutton-client/src/org/bigbluebutton/main/views/MainToolbar.mxml
@@ -33,6 +33,7 @@ with BigBlueButton; if not, see <http://www.gnu.org/licenses/>.
 	<mate:Listener type="{SettingsEvent.SETTINGS_MODULE_LOADED}" method="showSettingsButton" />
 	<mate:Listener type="{ShortcutEvent.REMOTE_OPEN_SHORTCUT_WIN}" method="remoteShortcutClick" />
 	<mate:Listener type="{ShortcutEvent.LOGOUT}" method="remoteLogout" />
+	<mate:Listener type="{ShortcutEvent.FOCUS_SHORTCUT_BUTTON}" method="focusShortcutButton" />
 	
 	
 	<mx:Script>
@@ -227,6 +228,10 @@ with BigBlueButton; if not, see <http://www.gnu.org/licenses/>.
 				var d:Dispatcher = new Dispatcher();
 				d.dispatchEvent(new ShortcutEvent(ShortcutEvent.OPEN_SHORTCUT_WIN));
 			}
+			
+			private function focusShortcutButton(e:ShortcutEvent):void{
+				shortcutKeysBtn.setFocus();
+			}
 
 		]]>
 	</mx:Script>	
diff --git a/bigbluebutton-client/src/org/bigbluebutton/main/views/ShortcutHelpWindow.mxml b/bigbluebutton-client/src/org/bigbluebutton/main/views/ShortcutHelpWindow.mxml
index 64a5aff462..2f6e8d2871 100755
--- a/bigbluebutton-client/src/org/bigbluebutton/main/views/ShortcutHelpWindow.mxml
+++ b/bigbluebutton-client/src/org/bigbluebutton/main/views/ShortcutHelpWindow.mxml
@@ -28,10 +28,13 @@ with BigBlueButton; if not, see <http://www.gnu.org/licenses/>.
 		   title="{ResourceUtil.getInstance().getString('bbb.shortcuthelp.title')}" >
 		   
 	<mate:Listener type="{LocaleChangeEvent.LOCALE_CHANGED}" method="localeChanged" />
+	<mate:Listener type="{MDIWindowEvent.CLOSE}" method="focusButton" />
 	
 	<mx:Script>
 		<![CDATA[
 			import flash.events.Event;
+			import com.asfusion.mate.events.Dispatcher;
+			import flexlib.mdi.events.MDIWindowEvent;
 			
 			import mx.collections.ArrayCollection;
 			import mx.collections.ArrayList;
@@ -94,6 +97,12 @@ with BigBlueButton; if not, see <http://www.gnu.org/licenses/>.
 				changeArray();
 			}
 			
+			public override function close(event:MouseEvent = null):void{
+				var dispatcher:Dispatcher = new Dispatcher();
+				dispatcher.dispatchEvent(new ShortcutEvent(ShortcutEvent.FOCUS_SHORTCUT_BUTTON));
+				super.close(event);
+			}
+			
 			private function changeArray():void {
 				shownKeys = new ArrayCollection();
 				switch(categories.selectedIndex) {
@@ -179,6 +188,16 @@ with BigBlueButton; if not, see <http://www.gnu.org/licenses/>.
 				focusManager.setFocus(keyList);
 				keyList.drawFocus(true);
 			}
+			
+			public function focusHead():void {
+				focusManager.setFocus(titleBarOverlay);
+			}
+			
+			public function focusButton(e:MDIWindowEvent):void{
+				LogUtil.debug("Caught close event from window");
+				var dispatcher:Dispatcher = new Dispatcher();
+				dispatcher.dispatchEvent(new ShortcutEvent(ShortcutEvent.FOCUS_SHORTCUT_BUTTON));
+			}
 		]]>
 	</mx:Script>