mirrors/element-web
1
0
Fork 0
mirror of https://github.com/element-hq/element-web synced 2024-11-28 12:28:50 +03:00
Code Issues Projects Releases Packages Wiki Activity

Merge pull request #1960 from matrix-org/t3chguy/tag_panel_a11y

Browse source 
improve tag panel accessibility and remove a no-op dispatch
This commit is contained in:
David Baker 2018-06-14 10:56:03 +01:00 committed by GitHub
commit 84a9c6a824
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 14 additions and 10 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

14
src/components/structures/TagPanel.js
View file

@ -84,7 +84,10 @@ const TagPanel = React.createClass({
}, },
onMouseDown(e) { onMouseDown(e) {
// only dispatch if its not a no-op
if (this.state.selectedTags.length > 0) {
dis.dispatch({action: 'deselect_tags'}); 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;
if (this.state.selectedTags.length > 0) {
clearButton = <AccessibleButton className="mx_TagPanel_clearButton" onClick={this.onClearFilterClick}>
<TintableSvg src="img/icons-close.svg" width="24" height="24" <TintableSvg src="img/icons-close.svg" width="24" height="24"
alt={_t("Clear filter")} alt={_t("Clear filter")}
title={_t("Clear filter")} title={_t("Clear filter")}
/> : />
<div />; </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"