improve tag panel accessibility and remove a no-op dispatch

Signed-off-by: Michael Telatynski <7t3chguy@gmail.com>
This commit is contained in:
Michael Telatynski 2018-06-13 16:17:26 +01:00
parent e858cf3bcb
commit 75abb415d4
No known key found for this signature in database
GPG key ID: 3F879DA5AD802A5E

View file

@ -84,7 +84,10 @@ const TagPanel = React.createClass({
}, },
onMouseDown(e) { onMouseDown(e) {
dis.dispatch({action: 'deselect_tags'}); // only dispatch if its not a no-op
if (this.state.selectedTags.length > 0) {
dis.dispatch({action: 'deselect_tags'});
}
}, },
onCreateGroupClick(ev) { onCreateGroupClick(ev) {
@ -113,17 +116,18 @@ const TagPanel = React.createClass({
/>; />;
}); });
const clearButton = this.state.selectedTags.length > 0 ? let clearButton;
<TintableSvg src="img/icons-close.svg" width="24" height="24" if (this.state.selectedTags.length > 0) {
alt={_t("Clear filter")} clearButton = <AccessibleButton className="mx_TagPanel_clearButton" onClick={this.onClearFilterClick}>
title={_t("Clear filter")} <TintableSvg src="img/icons-close.svg" width="24" height="24"
/> : alt={_t("Clear filter")}
<div />; title={_t("Clear filter")}
/>
</AccessibleButton>;
}
return <div className="mx_TagPanel"> return <div className="mx_TagPanel">
<AccessibleButton className="mx_TagPanel_clearButton" onClick={this.onClearFilterClick}> { clearButton }
{ clearButton }
</AccessibleButton>
<div className="mx_TagPanel_divider" /> <div className="mx_TagPanel_divider" />
<GeminiScrollbarWrapper <GeminiScrollbarWrapper
className="mx_TagPanel_scroller" className="mx_TagPanel_scroller"