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

VS Code: "Show Diff" ghost text is not clickable. #3509

Closed
jay-fibi opened this issue Mar 22, 2024 · 2 comments · Fixed by #3537
Closed

VS Code: "Show Diff" ghost text is not clickable. #3509

jay-fibi opened this issue Mar 22, 2024 · 2 comments · Fixed by #3537
Assignees
Labels

Comments

@jay-fibi
Copy link
Collaborator

jay-fibi commented Mar 22, 2024

Version

v1.10.0

Describe the bug

Platform: MacOS

Steps to reproduce:

  1. Have Cody plugin installed in your VS Code editor
  2. Login to Cody with Pro user
  3. Open any source code file, select a code snippet
  4. Right click and select "Cody > Document Code"
  5. Wait for Cody to generate document for the selected code snippet
  6. Try to click on "Show Diff" ghost text

Observed behavior:

"Show Diff" text is not clickable

Expected behavior

"Show Diff" ghost text should be clickable and display the diff.

Additional context

Screenshot:

Screenshot 2024-03-22 at 8 12 34 PM

@jay-fibi jay-fibi added bug Something isn't working clients/vscode fibilabs-qa labels Mar 22, 2024
@github-actions github-actions bot added the cody label Mar 22, 2024
@jay-fibi jay-fibi changed the title VS Code: "Show diff" ghost text is not clickable. VS Code: "Show Diff" ghost text is not clickable. Mar 22, 2024
@umpox umpox self-assigned this Mar 22, 2024
@umpox
Copy link
Contributor

umpox commented Mar 22, 2024

We no longer calculate diffs for outputs where we're only inserting new code.

We should hide this CTA and instead show the diff directly in the document (add the git diff added background)

@umpox
Copy link
Contributor

umpox commented Mar 25, 2024

PR for quick fix: #3537

instead show the diff directly in the document (add the git diff added background)

Will do this in a follow up PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants