Skip to content

Commit 880cb4a

Browse files
authored
Merge pull request modelcontextprotocol#322 from Kavyapriya-1804/feat-display-section-cleared-on-clear-button-hit
feat - Display section reset on clear button hit
2 parents 78cd701 + 9996be1 commit 880cb4a

File tree

2 files changed

+20
-5
lines changed

2 files changed

+20
-5
lines changed

client/src/components/PromptsTab.tsx

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,7 @@ const PromptsTab = ({
4343
clearPrompts: () => void;
4444
getPrompt: (name: string, args: Record<string, string>) => void;
4545
selectedPrompt: Prompt | null;
46-
setSelectedPrompt: (prompt: Prompt) => void;
46+
setSelectedPrompt: (prompt: Prompt | null) => void;
4747
handleCompletion: (
4848
ref: PromptReference | ResourceReference,
4949
argName: string,
@@ -89,7 +89,10 @@ const PromptsTab = ({
8989
<ListPane
9090
items={prompts}
9191
listItems={listPrompts}
92-
clearItems={clearPrompts}
92+
clearItems={() => {
93+
clearPrompts();
94+
setSelectedPrompt(null);
95+
}}
9396
setSelectedItem={(prompt) => {
9497
setSelectedPrompt(prompt);
9598
setPromptArgs({});

client/src/components/ResourcesTab.tsx

Lines changed: 15 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -104,7 +104,6 @@ const ResourcesTab = ({
104104
if (selectedTemplate) {
105105
const uri = fillTemplate(selectedTemplate.uriTemplate, templateValues);
106106
readResource(uri);
107-
setSelectedTemplate(null);
108107
// We don't have the full Resource object here, so we create a partial one
109108
setSelectedResource({ uri, name: uri } as Resource);
110109
}
@@ -116,7 +115,13 @@ const ResourcesTab = ({
116115
<ListPane
117116
items={resources}
118117
listItems={listResources}
119-
clearItems={clearResources}
118+
clearItems={() => {
119+
clearResources();
120+
// Condition to check if selected resource is not resource template's resource
121+
if (!selectedTemplate) {
122+
setSelectedResource(null);
123+
}
124+
}}
120125
setSelectedItem={(resource) => {
121126
setSelectedResource(resource);
122127
readResource(resource.uri);
@@ -139,7 +144,14 @@ const ResourcesTab = ({
139144
<ListPane
140145
items={resourceTemplates}
141146
listItems={listResourceTemplates}
142-
clearItems={clearResourceTemplates}
147+
clearItems={() => {
148+
clearResourceTemplates();
149+
// Condition to check if selected resource is resource template's resource
150+
if (selectedTemplate) {
151+
setSelectedResource(null);
152+
}
153+
setSelectedTemplate(null);
154+
}}
143155
setSelectedItem={(template) => {
144156
setSelectedTemplate(template);
145157
setSelectedResource(null);

0 commit comments

Comments
 (0)