fix insight icons

SatcherInstitute · Jan 10, 2025 · 7fcb0ee · 7fcb0ee
1 parent bb4c20a
commit 7fcb0ee
Show file tree

Hide file tree

Showing 3 changed files with 1 addition and 8 deletions.
diff --git a/frontend/src/cards/CardWrapper.tsx b/frontend/src/cards/CardWrapper.tsx
@@ -58,7 +58,6 @@ function CardWrapper(props: {
         return (
           <article
             className={`relative m-2 rounded-sm bg-white p-3 shadow-raised ${props.className}`}
-            tabIndex={-1}
           >
             {shouldShowInsightDisplay && (
               <InsightDisplay

diff --git a/frontend/src/cards/ui/InsightDisplay.tsx b/frontend/src/cards/ui/InsightDisplay.tsx
@@ -58,6 +58,7 @@ const InsightDisplay: React.FC<InsightDisplayProps> = ({
   return (
     <>
       <IconButton
+        aria-label={insight ? 'Clear insight' : 'Generate insight'}
         onClick={insight ? handleClearInsight : handleGenerateInsight}
         className='absolute top-2 right-2 z-10'
         disabled={isGeneratingInsight}

diff --git a/frontend/src/styles/HetComponents/HetTermUnderline.tsx b/frontend/src/styles/HetComponents/HetTermUnderline.tsx
@@ -9,7 +9,6 @@ interface HetTermUnderlineProps {
 export default function HetTermUnderline({
   children,
   className,
-  tabIndex = 0,
 }: HetTermUnderlineProps) {
   const spanRef = useRef<HTMLSpanElement>(null)
   const [isVisible, setIsVisible] = useState(false)
@@ -69,12 +68,6 @@ export default function HetTermUnderline({
         backgroundSize: '0% 8px',
         backgroundRepeat: 'no-repeat',
       }}
-      tabIndex={tabIndex}
-      onKeyDown={(e) => {
-        if (e.key === 'Enter' || e.key === ' ') {
-          e.preventDefault()
-        }
-      }}
     >
       {children}
     </span>