Skip to content

Autocomplete Hinter Parity#4043

Open
raclim wants to merge 7 commits intodevelop-codemirror-v6from
autocomplete-hinter-parity
Open

Autocomplete Hinter Parity#4043
raclim wants to merge 7 commits intodevelop-codemirror-v6from
autocomplete-hinter-parity

Conversation

@raclim
Copy link
Copy Markdown
Collaborator

@raclim raclim commented Mar 27, 2026

Description:

This PR makes progress towards parity with the current autocomplete hinter (styling, context-awareness, ast analyzer) through the following changes:

  • Adding back kind labels and ghost text previews to hinter (update-p5-hinter.js, p5-hinter.js, p5CompletionPreview.js)
  • Refactor contextaware and analyzer utils to adapt to CM6 and reattach them to hinter (contextAwareHinter.js, getContext.js, p5CodeAstAnalyzer.js, stateUtils,js, p5JavaScript.js)
  • Adding in custom keyboard behavior and UI for the hinter dropdown (stateUtils,js, _hints.scss)

Demo:

CM6 Autocomplete Hinter
Screenshot 2026-03-26 at 9 01 51 PM

CM5 Autocomplete Hinter
Screenshot 2026-03-26 at 9 02 02 PM

I have verified that this pull request:

  • has no linting errors (npm run lint)
  • has no test errors (npm run test)
  • has no typecheck errors (npm run typecheck)
  • is from a uniquely-named feature branch and is up to date with the develop branch.
  • is descriptively named and links to an issue number, i.e. Fixes #123
  • meets the standards outlined in the accessibility guidelines

@raclim raclim requested a review from khanniie March 27, 2026 01:11
@raclim
Copy link
Copy Markdown
Collaborator Author

raclim commented Mar 27, 2026

Sorry it's a bit late! I tried to add back in most of the main features in the current hinter—I might've missed a few, but I think it could potentially be added in a follow up?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant