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

make sure we don't break any A or BUTTON keyboard handling

Browse source
This commit is contained in:
Bruno Windels 2019-07-17 16:53:12 +02:00
parent 10a74696b3
commit 4bde0c08ad
1 changed files with 5 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
src/components/structures/LoggedInView.js
View file

@ -374,12 +374,11 @@ const LoggedInView = React.createClass({
const focusedOnInputControl = targetTag === "INPUT" || const focusedOnInputControl = targetTag === "INPUT" ||
targetTag === "TEXTAREA" || targetTag === "TEXTAREA" ||
targetTag === "SELECT" || targetTag === "SELECT" ||
ev.target.getAttribute("contenteditable"); !!ev.target.getAttribute("contenteditable");
// we don't check for buttons that have focus here, const isClickShortcut = ev.target !== document.body &&
// because they should be using AccessibleButton which (ev.key === "Space" || ev.key === "Enter");
// calls stopPropagation on space or enter keydown, so
// that keydown event would never get here. if (!focusedOnInputControl && !isClickShortcut) {
if (!focusedOnInputControl) {
dis.dispatch({action: 'focus_composer'}, true); dis.dispatch({action: 'focus_composer'}, true);
ev.stopPropagation(); ev.stopPropagation();
// we should *not* preventDefault() here as // we should *not* preventDefault() here as