Skip to content

Commit ee5dccd

Browse files
committed
address comments from @LianaHus
1 parent 00c8d06 commit ee5dccd

File tree

3 files changed

+6
-10
lines changed

3 files changed

+6
-10
lines changed

libs/remix-ui/workspace/src/lib/components/file-explorer-hovericons.tsx

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ export type FileHoverIconsProps = {
77
file: any
88
handleNewFolderOp?: any
99
handleNewFileOp?: any
10-
renamePathOp?: (path: string, newName: string) => void | Promise<void>
10+
renamePathOp?: (path: string, type: string) => void
1111
deletePathOp?: (path: string | string[]) => void | Promise<void>
1212
}
1313

@@ -27,7 +27,7 @@ export function FileHoverIcons(props: FileHoverIconsProps) {
2727
tooltipClasses="text-nowrap"
2828
>
2929
<span
30-
className="far fa-folder fa-1x remixui_icons_space remixui_icons"
30+
className="far fa-folder fa-1x mr-2 remixui_icons"
3131
onClick={async (e) => {
3232
e.stopPropagation()
3333
await props.handleNewFolderOp(props.file.path)
@@ -45,7 +45,7 @@ export function FileHoverIcons(props: FileHoverIconsProps) {
4545
tooltipClasses="text-nowrap"
4646
>
4747
<span
48-
className="far fa-file fa-1x remixui_icons remixui_icons_space"
48+
className="far fa-file fa-1x remixui_icons mr-2"
4949
onClick={async (e) => {
5050
e.stopPropagation()
5151
await props.handleNewFileOp(props.file.path)
@@ -66,7 +66,7 @@ export function FileHoverIcons(props: FileHoverIconsProps) {
6666
tooltipClasses="text-nowrap"
6767
>
6868
<span
69-
className="far fa-pen fa-1x remixui_icons remixui_icons_space"
69+
className="far fa-pen fa-1x remixui_icons mr-2"
7070
onClick={async (e) => {
7171
e.stopPropagation()
7272
await props.renamePathOp(props.file.path, props.file.type)
@@ -84,7 +84,7 @@ export function FileHoverIcons(props: FileHoverIconsProps) {
8484
tooltipClasses="text-nowrap"
8585
>
8686
<span
87-
className="far fa-trash fa-1x remixui_icons remixui_icons_space"
87+
className="far fa-trash fa-1x remixui_icons mr-2"
8888
onClick={async (e) => {
8989
e.stopPropagation()
9090
await props.deletePathOp(props.file.path)

libs/remix-ui/workspace/src/lib/components/flat-tree.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@ interface FlatTreeProps {
4141
createNewFile?: any
4242
createNewFolder?: any
4343
deletePath?: (path: string | string[]) => void | Promise<void>
44-
renamePath?: (path: string, type: string, isNew?: boolean) => void
44+
renamePath?: (path: string, type: string) => void
4545
editModeOn?: (path: string, type: string, isNew?: boolean) => void
4646
}
4747

libs/remix-ui/workspace/src/lib/css/file-explorer.css

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -69,7 +69,3 @@ ul {
6969
.remixui_icons:hover {
7070
color: var(--text);
7171
}
72-
73-
.remixui_icons_space {
74-
margin-right: 8px;
75-
}

0 commit comments

Comments
 (0)