Skip to content

Commit f67aaea

Browse files
committed
Fix to allow tick button to update a file name
1 parent 09d28c7 commit f67aaea

1 file changed

Lines changed: 10 additions & 18 deletions

File tree

src/components/ProjectName/ProjectName.jsx

Lines changed: 10 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -74,30 +74,21 @@ const ProjectName = ({
7474
}
7575
};
7676

77+
const handleOnBlur = (event) => {
78+
// If the blur event is triggered by clicking the tick button we want to make sure the target is clicked
79+
if (event.relatedTarget?.className?.includes("project-name__button")) {
80+
const target = event.relatedTarget ? event.relatedTarget : null;
81+
target.click();
82+
}
83+
resetName(event);
84+
};
85+
7786
useEffect(() => {
7887
if (isEditing) {
7988
nameInput.current.focus();
8089
}
8190
});
8291

83-
useEffect(() => {
84-
const handleClickOutside = (event) => {
85-
if (
86-
isEditing &&
87-
nameInput.current &&
88-
!nameInput.current.contains(event.target) &&
89-
tickButton.current &&
90-
!tickButton.current.contains(event.target)
91-
) {
92-
resetName(event);
93-
}
94-
};
95-
document.addEventListener("mousedown", handleClickOutside);
96-
return () => {
97-
document.removeEventListener("mousedown", handleClickOutside);
98-
};
99-
}, [isEditing, nameInput, tickButton, project, resetName]);
100-
10192
return (
10293
<>
10394
{showLabel && (
@@ -122,6 +113,7 @@ const ProjectName = ({
122113
value={name}
123114
disabled={!isEditing}
124115
onChange={handleOnChange}
116+
onBlur={handleOnBlur}
125117
/>
126118
) : (
127119
<div className="project-name__title">{name}</div>

0 commit comments

Comments
 (0)