-
Notifications
You must be signed in to change notification settings - Fork 1
Allozaur/mcp mvp #3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
…d per-chat overrides
tools/server/webui/src/lib/components/app/misc/DropdownMenuSearchable.svelte
Show resolved
Hide resolved
tools/server/webui/src/lib/markdown/resolve-attachment-images.ts
Outdated
Show resolved
Hide resolved
refactor: Cleanup refactor: Cleanup refactor: Cleanup
be151cc to
7f5284d
Compare
…l scroll bar behavior
allozaur
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Few remarks before thorough review of services + clients + stores
tools/server/webui/src/lib/components/app/misc/CollapsibleContentBlock.svelte
Outdated
Show resolved
Hide resolved
tools/server/webui/src/lib/components/app/mcp/McpServerCard/McpServerCardHeader.svelte
Show resolved
Hide resolved
tools/server/webui/src/lib/components/app/mcp/McpServerCard/McpServerCard.svelte
Outdated
Show resolved
Hide resolved
.../src/lib/components/app/chat/ChatForm/ChatFormPromptPicker/ChatFormPromptPickerHeader.svelte
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Improve UI of this component.
refactor: Cleanup refactor: Cleanup refactor: Cleanup
121e895 to
5bf1c86
Compare
| <div class="code-block-actions"> | ||
| <button | ||
| class="copy-code-btn" | ||
| title="Copy code" | ||
| aria-label="Copy code" | ||
| type="button" | ||
| onclick={() => copyCodeToClipboard(incompleteCodeBlock?.code ?? '')} | ||
| > | ||
| <svg | ||
| xmlns="http://www.w3.org/2000/svg" | ||
| width="16" | ||
| height="16" | ||
| viewBox="0 0 24 24" | ||
| fill="none" | ||
| stroke="currentColor" | ||
| stroke-width="2" | ||
| stroke-linecap="round" | ||
| stroke-linejoin="round" | ||
| ><rect width="14" height="14" x="8" y="8" rx="2" ry="2" /><path | ||
| d="M4 16c-1.1 0-2-.9-2-2V4c0-1.1.9-2 2-2h10c1.1 0 2 .9 2 2" | ||
| /></svg | ||
| > | ||
| </button> | ||
| {#if incompleteCodeBlock.language?.toLowerCase() === 'html'} | ||
| <button | ||
| class="preview-code-btn" | ||
| title="Preview code" | ||
| aria-label="Preview code" | ||
| type="button" | ||
| onclick={() => { | ||
| previewCode = incompleteCodeBlock?.code ?? ''; | ||
| previewLanguage = incompleteCodeBlock?.language ?? 'html'; | ||
| previewDialogOpen = true; | ||
| }} | ||
| > | ||
| <svg | ||
| xmlns="http://www.w3.org/2000/svg" | ||
| width="16" | ||
| height="16" | ||
| viewBox="0 0 24 24" | ||
| fill="none" | ||
| stroke="currentColor" | ||
| stroke-width="2" | ||
| stroke-linecap="round" | ||
| stroke-linejoin="round" | ||
| ><path | ||
| d="M2.062 12.345a1 1 0 0 1 0-.69C3.5 7.73 7.36 5 12 5s8.5 2.73 9.938 6.655a1 1 0 0 1 0 .69C20.5 16.27 16.64 19 12 19s-8.5-2.73-9.938-6.655" | ||
| /><circle cx="12" cy="12" r="3" /></svg | ||
| > | ||
| </button> | ||
| {/if} | ||
| </div> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- Code Block actions should be an individual component
- Make separate UI state for incomplete / complete code block
| {/if} | ||
| </div> | ||
|
|
||
| <CodePreviewDialog |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Rename to components/app/dialogs/DialogCodePreview.svelte
No description provided.