Merge "Make tools disable correctly"

This commit is contained in:
jenkins-bot 2013-11-13 00:54:38 +00:00 committed by Gerrit Code Review
commit f61227fc3c
2 changed files with 11 additions and 4 deletions

View file

@ -186,11 +186,14 @@ OO.ui.ToolGroup.prototype.onMouseOut = function ( e ) {
* @returns {OO.ui.Tool|null} Tool, `null` if none was found
*/
OO.ui.ToolGroup.prototype.getTargetTool = function ( e ) {
var $item = this.$( e.target ).closest( '.oo-ui-tool-link' );
var tool,
$item = this.$( e.target ).closest( '.oo-ui-tool-link' );
if ( $item.length ) {
return $item.parent().data( 'oo-ui-tool' );
tool = $item.parent().data( 'oo-ui-tool' );
}
return null;
return tool && !tool.isDisabled() ? tool : null;
};
/**

View file

@ -236,10 +236,14 @@
border-color: rgba(0,0,0,0.2);
}
.oo-ui-listToolGroup .oo-ui-tool.oo-ui-widget-disabled {
.oo-ui-listToolGroup .oo-ui-tool.oo-ui-widget-disabled .oo-ui-tool-link {
cursor: default;
}
.oo-ui-listToolGroup .oo-ui-tool.oo-ui-widget-disabled .oo-ui-tool-link .oo-ui-labeledElement-label {
color: #ccc;
}
.oo-ui-listToolGroup .oo-ui-tool.oo-ui-widget-disabled .oo-ui-tool-link .oo-ui-iconedElement-icon {
opacity: 0.2;
}