Skip to content

Commit afbe9f3

Browse files
committed
refactor(FoldersInput): improve folder display and cleanup code
Updated folder display logic to show folder names directly and enhanced styling for better readability. Cleaned up unused dependencies in the component.
1 parent e13ed51 commit afbe9f3

3 files changed

Lines changed: 16 additions & 20 deletions

File tree

package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
{
22
"name": "electron-agent-prompts",
33
"private": true,
4-
"version": "0.21.46",
4+
"version": "0.22.47",
55
"type": "module",
66
"main": "dist-main/app.js",
77
"author": "traeop",

src/renderer/components/FoldersInput/FoldersInput.tsx

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,7 @@ export const FoldersInput = ({ defaultFolders }: TFoldersInputProps) => {
5353
(folder: string) => () => {
5454
setFolders((current) => current.filter((item) => item !== folder));
5555
},
56-
[]
56+
[],
5757
);
5858

5959
return (
@@ -98,12 +98,10 @@ export const FoldersInput = ({ defaultFolders }: TFoldersInputProps) => {
9898
</Avatar>
9999
</ListItemAvatar>
100100
<ListItemText
101-
primary={folderName}
102-
secondary={folder}
101+
primary={folder}
103102
slotProps={{
104-
secondary: {
105-
noWrap: true,
106-
sx: { textOverflow: "ellipsis", overflow: "hidden" },
103+
primary: {
104+
sx: { wordBreak: "break-word" },
107105
},
108106
}}
109107
/>

src/renderer/conceptions/UpdateProject/components/UpdateProjectModal.test.tsx

Lines changed: 11 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ import { UpdateProjectModal } from "./UpdateProjectModal";
1717

1818
const createProject = (
1919
id: string,
20-
overrides?: Partial<TProject>
20+
overrides?: Partial<TProject>,
2121
): TProject => ({
2222
id,
2323
name: `Project ${id}`,
@@ -67,7 +67,7 @@ describe("UpdateProjectModal", () => {
6767
render(
6868
<Provider>
6969
<UpdateProjectModal onSuccess={vi.fn()} />
70-
</Provider>
70+
</Provider>,
7171
);
7272

7373
expect(screen.queryByTestId("update-project-form")).toBeNull();
@@ -81,7 +81,7 @@ describe("UpdateProjectModal", () => {
8181
<OpenModalWithProjectOnMount project={project}>
8282
<UpdateProjectModal onSuccess={vi.fn()} />
8383
</OpenModalWithProjectOnMount>
84-
</Provider>
84+
</Provider>,
8585
);
8686

8787
await waitFor(() => {
@@ -103,12 +103,12 @@ describe("UpdateProjectModal", () => {
103103
</OpenModalWithProjectOnMount>
104104
<ModalStateProbe />
105105
<ProjectProbe />
106-
</Provider>
106+
</Provider>,
107107
);
108108

109109
const form = await screen.findByTestId("update-project-form");
110110
const nameField = within(
111-
screen.getByTestId("update-project-name")
111+
screen.getByTestId("update-project-name"),
112112
).getByRole("textbox");
113113

114114
fireEvent.change(nameField, { target: { value: " Updated Name " } });
@@ -118,7 +118,6 @@ describe("UpdateProjectModal", () => {
118118
expect(updateProjectMock).toHaveBeenCalledWith({
119119
id: initialProject.id,
120120
name: "Updated Name",
121-
isGeneral: false,
122121
});
123122
});
124123

@@ -129,10 +128,10 @@ describe("UpdateProjectModal", () => {
129128
await waitFor(() => {
130129
expect(screen.queryByTestId("update-project-form")).toBeNull();
131130
expect(
132-
screen.getByTestId("update-project-modal-state")
131+
screen.getByTestId("update-project-modal-state"),
133132
).toHaveTextContent("false");
134133
expect(
135-
screen.getByTestId("update-project-modal-project")
134+
screen.getByTestId("update-project-modal-project"),
136135
).toHaveTextContent("none");
137136
});
138137
});
@@ -148,12 +147,12 @@ describe("UpdateProjectModal", () => {
148147
<OpenModalWithProjectOnMount project={project}>
149148
<UpdateProjectModal onSuccess={onSuccess} />
150149
</OpenModalWithProjectOnMount>
151-
</Provider>
150+
</Provider>,
152151
);
153152

154153
const form = await screen.findByTestId("update-project-form");
155154
const nameField = within(
156-
screen.getByTestId("update-project-name")
155+
screen.getByTestId("update-project-name"),
157156
).getByRole("textbox");
158157

159158
fireEvent.change(nameField, { target: { value: "New Name" } });
@@ -163,13 +162,12 @@ describe("UpdateProjectModal", () => {
163162
expect(updateProjectMock).toHaveBeenCalledWith({
164163
id: project.id,
165164
name: "New Name",
166-
isGeneral: false,
167165
});
168166
});
169167

170168
expect(onSuccess).not.toHaveBeenCalled();
171169
expect(
172-
await screen.findByTestId("update-project-form")
170+
await screen.findByTestId("update-project-form"),
173171
).toBeInTheDocument();
174172
});
175173

@@ -181,7 +179,7 @@ describe("UpdateProjectModal", () => {
181179
<OpenModalWithProjectOnMount project={project}>
182180
<UpdateProjectModal onSuccess={vi.fn()} />
183181
</OpenModalWithProjectOnMount>
184-
</Provider>
182+
</Provider>,
185183
);
186184

187185
await waitFor(() => {

0 commit comments

Comments
 (0)