tkmind/tkmind_go
1
0
Fork 0
Code Issues Pull Requests Actions 4 Packages Projects Releases Wiki Activity

tool count: fix colors and opacity (#2132)

Browse Source
This commit is contained in:
Lily Delalande
2025-04-10 14:22:10 -04:00
committed by GitHub
parent 5884fed4e6
commit df84c47c73
1 changed files with 9 additions and 27 deletions
Download Patch File Download Diff File Expand all files Collapse all files
ui/desktop/src/components/ToolCount.tsx
+9 -27
View File
@@ -1,10 +1,9 @@
import React, { useState, useEffect } from 'react'; import { useState, useEffect } from 'react';
import { getTools } from '../api'; import { getTools } from '../api';
import { ExclamationTriangleIcon } from '@radix-ui/react-icons'; import { ExclamationTriangleIcon } from '@radix-ui/react-icons';
import { HammerIcon } from 'lucide-react';
import { Popover, PopoverContent, PopoverTrigger } from './ui/popover'; import { Popover, PopoverContent, PopoverTrigger } from './ui/popover';
const SUGGESTED_MAX_TOOLS = 15; const SUGGESTED_MAX_TOOLS = 24;
export default function ToolCount() { export default function ToolCount() {
const [toolCount, setToolCount] = useState(null); const [toolCount, setToolCount] = useState(null);
@@ -37,24 +36,7 @@ export default function ToolCount() {
return <div>...</div>; return <div>...</div>;
} }
if (toolCount < SUGGESTED_MAX_TOOLS) { if (toolCount > SUGGESTED_MAX_TOOLS) {
return (
<div>
<Popover>
<PopoverTrigger asChild>
<button className="flex items-center justify-center p-0 border-0 bg-transparent cursor-pointer">
<HammerIcon size={16} />
</button>
</PopoverTrigger>
<PopoverContent className="p-3 w-auto" side="top">
<div className="space-y-1">
<p className="text-sm text-black dark:text-white">Tool count: {toolCount}</p>
</div>
</PopoverContent>
</Popover>
</div>
);
} else {
return ( return (
<div> <div>
<Popover> <Popover>
@@ -63,18 +45,18 @@ export default function ToolCount() {
<ExclamationTriangleIcon color="orange" /> <ExclamationTriangleIcon color="orange" />
</button> </button>
</PopoverTrigger> </PopoverTrigger>
<PopoverContent className="p-3" side="top"> <PopoverContent className="p-3 bg-orangit ge-500 " side="top">
<div className="space-y-2"> <div className="space-y-2">
<h4 className="text-sm font-medium">Warning: High Tool Count</h4> <p className="text-xs text-gray-300">
<p className="text-xs text-black dark:text-white"> Too many tools can degrade goose's performance. Consider turning unused extensions
Too many tools can degrade goose's performance. Consider turning a few extensions off. Tool count: {toolCount} (recommend: {SUGGESTED_MAX_TOOLS})
off.
</p> </p>
<p className="text-xs font-medium">Tool count: {toolCount}</p>
</div> </div>
</PopoverContent> </PopoverContent>
</Popover> </Popover>
</div> </div>
); );
} else {
return <div></div>;
} }
} }