Skip to content
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

show_completion_documentation = false removes slash command descriptions #20840

Open
1 task done
notpeter opened this issue Nov 18, 2024 · 2 comments
Open
1 task done
Labels
assistant AI feedback for Assistant (inline or panel) bug [core label] design papercut Small visual defect

Comments

@notpeter
Copy link
Member

Check for existing issues

  • Completed

Describe the bug / provide steps to reproduce it

Steps to reproduce:

  1. Add "show_completion_documentation": false, to your Zed User settings.
  2. Open an assistant panel.
  3. Type /

I don't know whether this is intended behavior.
I assume it just an accidental setting/style re-use.

Environment

Zed: v0.163.0 (Zed Nightly d92166f)
OS: macOS 14.7.0
Memory: 64 GiB
Architecture: aarch64

If applicable, add mockups / screenshots to help explain present your vision of the feature

Default:
Screenshot 2024-11-18 at 16 59 44

show_completion_documentation false in settings.
Screenshot 2024-11-18 at 16 59 51

If applicable, attach your Zed.log file to this issue.

N/A

@notpeter notpeter added bug [core label] triage Maintainer needs to classify the issue admin read Pending admin review design papercut Small visual defect assistant AI feedback for Assistant (inline or panel) and removed triage Maintainer needs to classify the issue admin read Pending admin review labels Nov 18, 2024
@mgsloan mgsloan self-assigned this Dec 8, 2024
@mgsloan
Copy link
Contributor

mgsloan commented Dec 8, 2024

I imagine this could be resolved via a special-case. However, a more consistent way for it to be resolved might be:

Not entirely sure if this is desirable / what the ramifications are. One issue with this is that slash completion appears to be active even when in other languages (such as a markdown code block that specifies a language).

@mgsloan mgsloan removed their assignment Dec 8, 2024
@nicholaswmin
Copy link

nicholaswmin commented Dec 19, 2024

I have this issue with code snippets - I think show_code_completion_documentation shouldn't affect the description; if you don't want a description simply don't define one:

  "await sleep": {
    "description": "add async delay", <-- // dont define this
    "prefix": ["sleep", "wait", "delay", "timeout"],  
    "body": "await new Promise(r => setTimeout(r, ${0}))"
  },

The completion documentation is super clunky for me but the inline-description is just as clunky as its turned on as when it's turned off

@mgsloan mgsloan self-assigned this Dec 21, 2024
@mgsloan mgsloan removed their assignment Jan 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
assistant AI feedback for Assistant (inline or panel) bug [core label] design papercut Small visual defect
Projects
None yet
Development

No branches or pull requests

3 participants