chore(a11y): enable jsx-a11y/click-events-have-key-events as error (#42009)

Co-authored-by: Claude Opus 4.8 <noreply@anthropic.com>
This commit is contained in:
Evan Rusackas
2026-07-15 15:49:45 -07:00
committed by GitHub
parent 1f786f1949
commit 1b0c6aaed3
47 changed files with 468 additions and 60 deletions

View File

@@ -16,6 +16,7 @@
* specific language governing permissions and limitations
* under the License.
*/
import { handleKeyboardActivation } from '@superset-ui/core';
import { css, styled, useTheme } from '@apache-superset/core/theme';
import { t } from '@apache-superset/core/translation';
import type { NodeRendererProps } from 'react-arborist';
@@ -186,6 +187,7 @@ const TreeNodeRenderer: React.FC<TreeNodeRendererProps> = ({
<span
className="col-copy-action"
onClick={e => e.stopPropagation()}
onKeyDown={e => e.stopPropagation()}
>
<ActionButton
label={`copy-col-${data.name}`}
@@ -215,6 +217,13 @@ const TreeNodeRenderer: React.FC<TreeNodeRendererProps> = ({
node.toggle();
}
}}
onKeyDown={handleKeyboardActivation(() => {
if (node.isLeaf) {
node.select();
} else {
node.toggle();
}
})}
>
<span
className="tree-node-icon"
@@ -239,6 +248,7 @@ const TreeNodeRenderer: React.FC<TreeNodeRendererProps> = ({
<div
className="side-action-container"
onClick={e => e.stopPropagation()}
onKeyDown={e => e.stopPropagation()}
>
{pinnedSchemas.has(schema) && (
<div className="action-static">
@@ -306,6 +316,7 @@ const TreeNodeRenderer: React.FC<TreeNodeRendererProps> = ({
<div
className="side-action-container"
onClick={e => e.stopPropagation()}
onKeyDown={e => e.stopPropagation()}
>
{isPinned && (
<div className="action-static">