Skip to content

Commit

Permalink
[monaco-graphql] remove newline trigger character (#3253)
Browse files Browse the repository at this point in the history
  • Loading branch information
acao authored Jun 22, 2023
1 parent 4a6f516 commit 0ae5e2d
Show file tree
Hide file tree
Showing 4 changed files with 7 additions and 11 deletions.
4 changes: 3 additions & 1 deletion .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,7 @@ coverage/
packages/graphiql/typedoc/
dist/
esm/
packages/*/out/
*.vsix
*.tsbuildinfo

Expand Down Expand Up @@ -41,8 +42,9 @@ packages/codemirror-graphql/*.map
!packages/codemirror-graphql/*.config.js

packages/graphiql/index.html
packages/graphiql/dev.html
packages/graphiql/analyzer.html
packages/graphiql/graphiql.*.js
packages/graphiql/graphiql*.js
packages/graphiql/*.css
packages/graphiql/*.map
packages/graphiql/cypress/screenshots/
5 changes: 4 additions & 1 deletion packages/monaco-graphql/src/languageFeatures.ts
Original file line number Diff line number Diff line change
Expand Up @@ -217,7 +217,10 @@ export class CompletionAdapter implements languages.CompletionItemProvider {
}

public get triggerCharacters(): string[] {
return [':', '$', '\n', ' ', '(', '@'];
// removing /n character for now until we can
// re-introduce the behavior in a programmatic,
// context-aware fashion
return [':', '$', ' ', '(', '@'];
}

async provideCompletionItems(
Expand Down
1 change: 0 additions & 1 deletion packages/vscode-graphql-execution/.gitignore

This file was deleted.

8 changes: 0 additions & 8 deletions packages/vscode-graphql/.gitignore

This file was deleted.

0 comments on commit 0ae5e2d

Please sign in to comment.