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

Test: Injected Text Adoption For Inline Suggestions #129383

Closed
2 tasks done
hediet opened this issue Jul 26, 2021 · 1 comment
Closed
2 tasks done

Test: Injected Text Adoption For Inline Suggestions #129383

hediet opened this issue Jul 26, 2021 · 1 comment

Comments

@hediet
Copy link
Member

hediet commented Jul 26, 2021

Refs

Complexity: 3

Create Issue

Create Issue


Inline Suggestions now use the new Injected Text feature of the core editor.
This fixes some issues that inline suggestions had and enables line wrapping of inline suggestions.

Prep

Install the copilot extension.

Testing

Play around with copilot and verify that

  • the cursor is never inside of an inline suggestion
  • When wrapping is enabled, long inline suggestions are wrapped correctly
  • Hover works correctly on ghost text
  • There are no weird UI artifacts.
@TylerLeonhardt
Copy link
Member

I've pretty much finished testing this item but since the steps are "use copilot" I'm not gonna unassign myself yet and will just give it another day of testing since it's easy to test.

@TylerLeonhardt TylerLeonhardt removed their assignment Jul 28, 2021
@github-actions github-actions bot locked and limited conversation to collaborators Sep 11, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants