Skip to content

Commit c1e9ff0

Browse files
authored
Fix model provider error message (#10923)
Address #7632 Gives a bit more space to the dialog height so that the error message can display without scrolling. Added a right margin so that the text won't extend into the button. Added bottom padding so that the dialog will scroll if the text does extend past the dialog height. <img width="610" height="509" alt="image" src="https://github.com/user-attachments/assets/2d22070c-7fab-4bfe-ab6f-5dae5a6aefa7" /> ### Release Notes #### New Features - N/A #### Bug Fixes - Fix model provider config dialog error message covered by dialog button ### QA Notes
1 parent a6628d5 commit c1e9ff0

File tree

2 files changed

+3
-1
lines changed

2 files changed

+3
-1
lines changed

src/vs/workbench/contrib/positronAssistant/browser/languageModelModalDialog.css

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -141,4 +141,6 @@
141141

142142
.language-model-error.error-msg {
143143
word-break: break-word;
144+
margin-right: 90px;
145+
padding-bottom: 10px;
144146
}

src/vs/workbench/contrib/positronAssistant/browser/languageModelModalDialog.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -341,7 +341,7 @@ const LanguageModelConfiguration = (props: React.PropsWithChildren<LanguageModel
341341
};
342342

343343
return <OKModalDialog
344-
height={450}
344+
height={500}
345345
okButtonTitle={(() => localize('positron.languageModelModalDialog.close', "Close"))()}
346346
renderer={props.renderer}
347347
title={(() => localize('positron.languageModelModalDialog.title', "Configure Language Model Providers"))()}

0 commit comments

Comments
 (0)