Limit scope of tab movement keys b=321128 p=Mnyromyr r=me (my tabbrowser.xml changes r=Mnyromyr) sr=jag

themes/classic/global/mac/tabbox.css

@@ -59,6 +59,9 @@ tab:hover:active > .tab-image-right {
 }
 
 /* when a tab is selected this is how it should look */
+tab[selected="true"] {
+  -moz-user-focus: normal;
+}
 
 tab[selected="true"] > .tab-image-left,
 tab[selected="true"]:hover:active > .tab-image-left {
@@ -77,6 +80,13 @@ tab[selected="true"]:hover:active > .tab-image-right {
   background		: url(chrome://global/skin/tabbox/tab-select-right.gif) no-repeat;
 }
 
+tab:focus {
+  outline: 3px solid -moz-mac-focusring;
+  outline-offset: -3px;
+  -moz-outline-radius-topleft:  5px;
+  -moz-outline-radius-topright: 5px;
+}
+
 .tab-border-top-left {
   -moz-appearance: dialog;
   background-color  : #DEDEDE;

xpfe/browser/resources/content/navigatorOverlay.xul

@@ -123,15 +123,6 @@
     <keyset id="viewZoomKeys"/>
     <keyset id="navigationKeys"/>  
     <keyset id="tasksKeys"/>
-
-    <!-- Tab reordering keys -->
-    <key id="key_moveTabLeft" keycode="VK_LEFT" modifiers="accel" oncommand="gBrowser.moveTabLeft()"/>
-    <key id="key_moveTabRight" keycode="VK_RIGHT" modifiers="accel" oncommand="gBrowser.moveTabRight()"/>
-    <key id="key_moveTabBackward" keycode="VK_UP" modifiers="accel" oncommand="gBrowser.moveTabBackward()"/>
-    <key id="key_moveTabForward" keycode="VK_DOWN" modifiers="accel" oncommand="gBrowser.moveTabForward()"/>
-    <key id="key_moveTabToStart" keycode="VK_HOME" modifiers="accel" oncommand="gBrowser.moveTabToStart()"/>
-    <key id="key_moveTabToEnd" keycode="VK_END" modifiers="accel" oncommand="gBrowser.moveTabToEnd()"/>
-
   </keyset>
 
   <commandset id="commands">

xpfe/global/resources/content/bindings/tabbrowser.xml

@@ -1839,6 +1839,13 @@
           }
         ]]>
       </handler>
+
+      <handler event="keypress" keycode="VK_LEFT" modifiers="accel" action="if (event.target == this) { this.moveTabLeft(); event.preventDefault(); }"/>
+      <handler event="keypress" keycode="VK_RIGHT" modifiers="accel" action="if (event.target == this) { this.moveTabRight();" event.preventDefault(); }/>
+      <handler event="keypress" keycode="VK_UP" modifiers="accel" action="if (event.target == this) { this.moveTabBackward();" event.preventDefault(); }/>
+      <handler event="keypress" keycode="VK_DOWN" modifiers="accel" action="if (event.target == this) { this.moveTabForward();" event.preventDefault(); }/>
+      <handler event="keypress" keycode="VK_HOME" modifiers="accel" action="if (event.target == this) { this.moveTabToStart();" event.preventDefault(); }/>
+      <handler event="keypress" keycode="VK_END" modifiers="accel" action="if (event.target == this) { this.moveTabToEnd();" event.preventDefault(); }/>
     </handlers>
   </binding>