mirror of
https://github.com/mozilla/gecko-dev.git
synced 2024-12-14 02:31:59 +00:00
Bug 679255 - Tab close button shouldn't be focusable. r=enn
This commit is contained in:
parent
316be0eb11
commit
4c959189fc
@ -16,12 +16,6 @@
|
||||
display: -moz-box;
|
||||
}
|
||||
|
||||
.tab-close-button[selected="true"] {
|
||||
/* Make this button focusable so clicking on it will not focus the tab while
|
||||
it's getting closed */
|
||||
-moz-user-focus: normal;
|
||||
}
|
||||
|
||||
.tab-label[pinned] {
|
||||
width: 0;
|
||||
margin-left: 0 !important;
|
||||
|
@ -4002,7 +4002,6 @@
|
||||
role="presentation"/>
|
||||
<xul:toolbarbutton anonid="close-button"
|
||||
xbl:inherits="fadein,pinned,selected"
|
||||
tabindex="-1"
|
||||
clickthrough="never"
|
||||
class="tab-close-button"/>
|
||||
</xul:hbox>
|
||||
@ -4040,25 +4039,14 @@
|
||||
<handler event="dragstart" phase="capturing">
|
||||
this.style.MozUserFocus = '';
|
||||
</handler>
|
||||
<handler event="mousedown" button="0" phase="capturing">
|
||||
<handler event="mousedown">
|
||||
<![CDATA[
|
||||
if (this.mOverCloseButton) {
|
||||
event.stopPropagation();
|
||||
}
|
||||
else if (this.selected) {
|
||||
if (this.selected) {
|
||||
this.style.MozUserFocus = 'ignore';
|
||||
this.clientTop; // just using this to flush style updates
|
||||
}
|
||||
]]>
|
||||
</handler>
|
||||
<handler event="mousedown" button="1">
|
||||
this.style.MozUserFocus = 'ignore';
|
||||
this.clientTop;
|
||||
</handler>
|
||||
<handler event="mousedown" button="2">
|
||||
this.style.MozUserFocus = 'ignore';
|
||||
this.clientTop;
|
||||
</handler>
|
||||
<handler event="mouseup">
|
||||
this.style.MozUserFocus = '';
|
||||
</handler>
|
||||
|
@ -1868,10 +1868,6 @@ richlistitem[type~="action"][actiontype="switchtab"] > .ac-url-box > .ac-action-
|
||||
-moz-image-region: rect(0, 16px, 16px, 0);
|
||||
}
|
||||
|
||||
.tab-close-button:focus {
|
||||
outline: none !important;
|
||||
}
|
||||
|
||||
/* Tab scrollbox arrow, tabstrip new tab and all-tabs buttons */
|
||||
|
||||
@media all and (-moz-touch-enabled) {
|
||||
|
Loading…
Reference in New Issue
Block a user