fix(auto-edit): fix the cody status bar with new suggestion mode #6691
Conversation
vscode/src/services/StatusBar.ts
Outdated
| async onSelect() { | ||
| // Show a QuickPick for the user to choose a new value. | ||
| const selection = await vscode.window.showQuickPick(requiredValues, { | ||
| placeHolder: `Current: “${currentValue}”. Pick a new mode...`, |
There was a problem hiding this comment.
Hi @toolmantim
I wanted to get your review on these string that we are showing to the user on the UI. Do you think these look okay or we should change them to something else ?
There was a problem hiding this comment.
This is how they currently look in the UI:
There was a problem hiding this comment.
All the existing settings in the "enable/disable features" are boolean… I think instead we add this in a section above it @hitesh-1997 & @umpox:
section label: "code suggestion settings"
icon: code (unless you find something better)
title: "Code Suggestion Mode (${current})"
description: "Choose how Cody suggests inline code changes"
q: Have we dropped the cog right hand action, for configuring autocomplete file types? We could add it to the next quickpick where you choose…
placeholder: "Choose an option"
section label: options
options:
- title: "Autocomplete"
description: "Suggest code completions for the rest of the line or block" - title: "Auto-edit"
description: "Suggest additions, changes and deletions around cursor based on file changes" - title: Disabled
description: "No code suggestions"
If autocomplete is enabled, add this extra section:
section label: settings
- title: "Autocomplete Settings"
description: "Open Cody autocomplete settings"
There was a problem hiding this comment.
Hi @toolmantim
Would it be possible to add a quick design snapshot ?
There was a problem hiding this comment.
Thanks a lot for the response @toolmantim
The new design looks really good, implemented the same :)
There was a problem hiding this comment.
LGTM from a code perspective, I'll leave @toolmantim to comment on the design but suggest we merge and fix forward :)
| @@ -127,13 +127,6 @@ export function isUserEligibleForAutoeditsFeature( | |||
| authStatus: AuthStatus, | |||
| productSubscription: UserProductSubscription | null | |||
| ): AutoeditsUserEligibilityInfo { | |||
| console.log({ | |||
Context
Cody Status bar for
autocompleteis broken right now with the introduction of new suggestion modes.autocomplete/off/auto-edit(if applicable).Auto-editis only shown if the user is eligible for the feature. See in the demo for the free account, we don't show this option.Test plan
Loom.Message.-.18.January.2025.mp4