Skip to content

Commit 8aa6692

Browse files
committed
feat: add keys to reference via index
1 parent bbb5078 commit 8aa6692

5 files changed

Lines changed: 18 additions & 16 deletions

File tree

src/components/experience/ExperienceItem.tsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -29,8 +29,8 @@ export default function ExperienceItem(props: ExperienceItemProps) {
2929
</div>
3030

3131
<ul className="list-disc pl-4">
32-
{props.description.map((content) => (
33-
<li className="pl-2 py-1.5">
32+
{props.description.map((content, index) => (
33+
<li className="pl-2 py-1.5" key={index}>
3434
{content}
3535
</li>
3636
))}

src/components/header/HeaderListItem.tsx

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,12 @@
11
interface HeaderListItemProps {
22
name: string;
33
id: string;
4+
index: number;
45
}
56

67
export default function HeaderListItem(props: HeaderListItemProps) {
78
return (
8-
<li className="hover:underline hover:scale-110 text-xl hidden sm:block">
9+
<li className="hover:underline hover:scale-110 text-xl hidden sm:block" key={props.index}>
910
<a href={`#${props.id}`}>
1011
{props.name}
1112
</a>

src/components/header/HeaderListItemMobile.tsx

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,11 +6,12 @@ interface HeaderListItemMobileProps {
66
id: string;
77
isOpen: boolean;
88
setIsOpen: Dispatch<SetStateAction<boolean>>;
9+
index: number;
910
}
1011

1112
export default function HeaderListItemMobile(props: HeaderListItemMobileProps) {
1213
return (
13-
<li className="hover:underline hover:scale-110 text-xl py-3 text-center">
14+
<li className="hover:underline hover:scale-110 text-xl py-3 text-center" key={props.index}>
1415
<a href={`#${props.id}`} onClick={() => props.setIsOpen(!props.isOpen)}>
1516
{props.name}
1617
</a>

src/components/header/NavbarContent.tsx

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -4,12 +4,12 @@ export default function NavbarContent() {
44
return (
55
<>
66
<ul className="gap-8 justify-center hidden md:flex py-8">
7-
<HeaderListItem name="Home" id="home" />
8-
<HeaderListItem name="Experience" id="experience" />
9-
<HeaderListItem name="Skills" id="skills" />
10-
<HeaderListItem name="Education" id="education" />
11-
<HeaderListItem name="Projects" id="projects" />
12-
<HeaderListItem name="Open Source" id="opensource" />
7+
<HeaderListItem name="Home" id="home" index={0}/>
8+
<HeaderListItem name="Experience" id="experience" index={1} />
9+
<HeaderListItem name="Skills" id="skills" index={2} />
10+
<HeaderListItem name="Education" id="education" index={3} />
11+
<HeaderListItem name="Projects" id="projects" index={4} />
12+
<HeaderListItem name="Open Source" id="opensource" index={5} />
1313
</ul>
1414
</>
1515
);

src/components/header/NavbarContentMobile.tsx

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -22,12 +22,12 @@ export default function NavbarContentMobile() {
2222
Pages
2323
</div>
2424
<ul className="flex-row justify-center">
25-
<HeaderListItemMobile name="Home" id="home" isOpen={isMenuOpen} setIsOpen={setIsMenuOpen} />
26-
<HeaderListItemMobile name="Experience" id="experience" isOpen={isMenuOpen} setIsOpen={setIsMenuOpen} />
27-
<HeaderListItemMobile name="Skills" id="skills" isOpen={isMenuOpen} setIsOpen={setIsMenuOpen} />
28-
<HeaderListItemMobile name="Education" id="education" isOpen={isMenuOpen} setIsOpen={setIsMenuOpen} />
29-
<HeaderListItemMobile name="Projects" id="projects" isOpen={isMenuOpen} setIsOpen={setIsMenuOpen} />
30-
<HeaderListItemMobile name="Open Source" id="opensource" isOpen={isMenuOpen} setIsOpen={setIsMenuOpen} />
25+
<HeaderListItemMobile name="Home" id="home" isOpen={isMenuOpen} setIsOpen={setIsMenuOpen} index={0}/>
26+
<HeaderListItemMobile name="Experience" id="experience" isOpen={isMenuOpen} setIsOpen={setIsMenuOpen} index={1}/>
27+
<HeaderListItemMobile name="Skills" id="skills" isOpen={isMenuOpen} setIsOpen={setIsMenuOpen} index={2}/>
28+
<HeaderListItemMobile name="Education" id="education" isOpen={isMenuOpen} setIsOpen={setIsMenuOpen} index={3}/>
29+
<HeaderListItemMobile name="Projects" id="projects" isOpen={isMenuOpen} setIsOpen={setIsMenuOpen} index={4}/>
30+
<HeaderListItemMobile name="Open Source" id="opensource" isOpen={isMenuOpen} setIsOpen={setIsMenuOpen} index={5}/>
3131
</ul>
3232
</div>
3333
)}

0 commit comments

Comments
 (0)