Skip to content

feat: update the phantom while typing#62

Draft
timfjord wants to merge 1 commit into
masterfrom
handle-text-change
Draft

feat: update the phantom while typing#62
timfjord wants to merge 1 commit into
masterfrom
handle-text-change

Conversation

@timfjord

@timfjord timfjord commented Oct 5, 2022

Copy link
Copy Markdown
Collaborator

Closes #52

This PR tries to emulate VSCode when the phantom(ghost) text is updated while typing(without triggering an extra request).
Unfortunately, due to the phantom API limitations, it is currently not possible(see the attached video)

phantom.mov

The main problem is that the cursor always jumps when updating the phantom in real time.

@timfjord

timfjord commented Oct 5, 2022

Copy link
Copy Markdown
Collaborator Author

I opened it just to have some reference but, in general, this PR should be merged only when the problem with the cursor is fixed(see sublimehq/sublime_text#5536 and #54)

@jfcherng

jfcherng commented Oct 5, 2022

Copy link
Copy Markdown
Collaborator

I feel, at this moment, the UX in the video is worse than without this PR.

@timfjord

timfjord commented Oct 5, 2022

Copy link
Copy Markdown
Collaborator Author

Just in case, here is how it works in VSCode

vscode.mov

@timfjord

timfjord commented Oct 5, 2022

Copy link
Copy Markdown
Collaborator Author

Yes, it is horrible. That's why we shouldn't merge this one.

We could close it, as I opened it purely to have some reference

@jfcherng

jfcherng commented Oct 5, 2022

Copy link
Copy Markdown
Collaborator

We could close it, as I opened it purely to have some reference

Keep it open is fine to help us remember this 😄

@jfcherng jfcherng added the enhancement New feature or request label Jun 21, 2024
@jfcherng
jfcherng force-pushed the master branch 2 times, most recently from 10ee440 to 6c84c74 Compare July 4, 2024 09:07
@jfcherng
jfcherng force-pushed the master branch 4 times, most recently from b326282 to bd39fac Compare August 2, 2024 21:19
@jfcherng
jfcherng force-pushed the master branch 3 times, most recently from 3a32507 to 028ea27 Compare September 26, 2025 20:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Handle text changes when phantom completion is visible

2 participants