change cursor:pointer iff clicking TagPanel background does something

Signed-off-by: Michael Telatynski <7t3chguy@gmail.com>
pull/21833/head
Michael Telatynski 2018-06-14 11:42:21 +01:00
parent c811c30c0d
commit 3b501137a5
No known key found for this signature in database
GPG Key ID: 3F879DA5AD802A5E
2 changed files with 14 additions and 4 deletions

View File

@ -17,7 +17,6 @@ limitations under the License.
.mx_TagPanel {
flex: 0 0 60px;
background-color: $tertiary-accent-color;
cursor: pointer;
display: flex;
flex-direction: column;
@ -25,6 +24,10 @@ limitations under the License.
justify-content: space-between;
}
.mx_TagPanel_items_selected {
cursor: pointer;
}
.mx_TagPanel .mx_TagPanel_clearButton_container {
/* Constant height within flex mx_TagPanel */
height: 70px;

View File

@ -1,5 +1,5 @@
/*
Copyright 2017 New Vector Ltd.
Copyright 2017, 2018 New Vector Ltd.
Licensed under the Apache License, Version 2.0 (the "License");
you may not use this file except in compliance with the License.
@ -26,6 +26,7 @@ import dis from '../../dispatcher';
import { _t } from '../../languageHandler';
import { Droppable } from 'react-beautiful-dnd';
import classNames from 'classnames';
const TagPanel = React.createClass({
displayName: 'TagPanel',
@ -116,8 +117,10 @@ const TagPanel = React.createClass({
/>;
});
const itemsSelected = this.state.selectedTags.length > 0;
let clearButton;
if (this.state.selectedTags.length > 0) {
if (itemsSelected) {
clearButton = <AccessibleButton className="mx_TagPanel_clearButton" onClick={this.onClearFilterClick}>
<TintableSvg src="img/icons-close.svg" width="24" height="24"
alt={_t("Clear filter")}
@ -126,7 +129,11 @@ const TagPanel = React.createClass({
</AccessibleButton>;
}
return <div className="mx_TagPanel">
const classes = classNames('mx_TagPanel', {
mx_TagPanel_items_selected: itemsSelected,
});
return <div className={classes}>
<div className="mx_TagPanel_clearButton_container">
{ clearButton }
</div>