Skip to content

Commit

Permalink
Merge pull request #2339 from posit-dev/dotnomad/correct-new-details
Browse files Browse the repository at this point in the history
Only show select existing text if options exist
  • Loading branch information
dotNomad authored Oct 2, 2024
2 parents 28034e3 + 2d4d370 commit 75b6d95
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -126,7 +126,9 @@ export async function selectNewOrExistingConfig(
configFileListItems.push({
iconPath: new ThemeIcon("plus"),
label: createNewConfigurationLabel,
detail: "(or pick one of the existing deployments below)",
detail: configurations.length
? "(or pick one of the existing configurations below)"
: undefined,
picked: configurations.length ? false : true,
});

Expand Down
4 changes: 3 additions & 1 deletion extensions/vscode/src/views/homeView.ts
Original file line number Diff line number Diff line change
Expand Up @@ -1121,7 +1121,9 @@ export class HomeViewProvider implements WebviewViewProvider, Disposable {
deploymentQuickPickList.push({
iconPath: new ThemeIcon("plus"),
label: createNewDeploymentLabel,
detail: "(or pick one of the existing deployments below)", // we're forcing a blank here, just to maintain height of selection
detail: includedContentRecords.length
? "(or pick one of the existing deployments below)"
: undefined,
lastMatch: includedContentRecords.length ? false : true,
});

Expand Down

0 comments on commit 75b6d95

Please sign in to comment.