auto-claude: subtask-2-1 - Move iconMap outside component and memoize event handlers

This commit is contained in:
2026-01-25 06:34:00 +01:00
parent 0f5bba66a8
commit 9bca232ca7
+18 -4
View File
@@ -1,6 +1,6 @@
'use client'; 'use client';
import { useState, useEffect } from 'react'; import { useState, useEffect, useCallback } from 'react';
import { useTranslations } from 'next-intl'; import { useTranslations } from 'next-intl';
import { motion } from 'framer-motion'; import { motion } from 'framer-motion';
import { Database, Server, Store, Code2, Box } from 'lucide-react'; import { Database, Server, Store, Code2, Box } from 'lucide-react';
@@ -15,6 +15,7 @@ interface SkillGroup {
items: SkillItem[]; items: SkillItem[];
} }
// Move iconMap outside component for better performance
const iconMap: Record<string, React.ReactNode> = { const iconMap: Record<string, React.ReactNode> = {
'Python': <Code2 className="w-8 h-8" />, 'Python': <Code2 className="w-8 h-8" />,
'Server': <Server className="w-8 h-8" />, 'Server': <Server className="w-8 h-8" />,
@@ -32,6 +33,19 @@ const Skills = () => {
const [selectedSkill, setSelectedSkill] = useState<string | null>(null); const [selectedSkill, setSelectedSkill] = useState<string | null>(null);
const [allSkills, setAllSkills] = useState<SkillItem[]>([]); const [allSkills, setAllSkills] = useState<SkillItem[]>([]);
// Memoized event handlers
const handleHoverStart = useCallback((skillName: string) => {
setSelectedSkill(skillName);
}, []);
const handleHoverEnd = useCallback(() => {
setSelectedSkill(null);
}, []);
const handleClick = useCallback((skillName: string) => {
setSelectedSkill(prev => prev === skillName ? null : skillName);
}, []);
useEffect(() => { useEffect(() => {
try { try {
const skillGroups = t.raw('skillGroups') as SkillGroup[]; const skillGroups = t.raw('skillGroups') as SkillGroup[];
@@ -58,8 +72,8 @@ const Skills = () => {
className="space-y-2" className="space-y-2"
initial={{ opacity: 0 }} initial={{ opacity: 0 }}
animate={{ opacity: 1 }} animate={{ opacity: 1 }}
onHoverStart={() => setSelectedSkill(skill.name)} onHoverStart={() => handleHoverStart(skill.name)}
onHoverEnd={() => setSelectedSkill(null)} onHoverEnd={handleHoverEnd}
> >
<div className="flex justify-between items-center"> <div className="flex justify-between items-center">
<div className="flex flex-col"> <div className="flex flex-col">
@@ -106,7 +120,7 @@ const Skills = () => {
selectedSkill === skill.name ? 'ring-2 ring-zinc-500' : '' selectedSkill === skill.name ? 'ring-2 ring-zinc-500' : ''
}`} }`}
whileHover={{ scale: 1.05 }} whileHover={{ scale: 1.05 }}
onClick={() => setSelectedSkill(skill.name === selectedSkill ? null : skill.name)} onClick={() => handleClick(skill.name)}
role="button" role="button"
aria-pressed={selectedSkill === skill.name} aria-pressed={selectedSkill === skill.name}
> >