graphql-language-service icon indicating copy to clipboard operation
graphql-language-service copied to clipboard

Published 20 hours ago • verified publisher icon graphql

lsp: textDocument/documentSymbol support

Open stephen opened this issue 7 years ago • 8 comments

This PR adds support for textDocument/documentSymbol to the lsp-compatible server.

Here's a screenshot in vscode: outline

stephen avatar Dec 26 '17 23:12 stephen

@asiandrummer could you ptal at this and #196?

stephen avatar Jan 03 '18 17:01 stephen

Yeah sorry - I'll review this now

asiandrummer avatar Jan 03 '18 19:01 asiandrummer

This is awesome, merge pls? 🙏

lostplan avatar Feb 06 '18 18:02 lostplan

@asiandrummer how can i convince y'all to take a look at this again? Happy to rebase and sort out merge conflicts if that's all it needs.

stephen avatar Apr 26 '18 05:04 stephen

I'm happy to take a look (next week at the soonest)

lostplan avatar Apr 26 '18 14:04 lostplan

Looks good to me. @stephen if you could rebase etc then I think it should be merged.

lostplan avatar Jun 11 '18 12:06 lostplan

Any update? This would be really nice to have. @stephen

nathanchapman avatar Jan 18 '19 21:01 nathanchapman

well well well, time to recreate this for the new monorepo at https://github.com/graphql/graphiql!

some of this is gonna have to be re-done in TS but the server is still made with flow. can't wait to have this in monaco -> new graphiql too :D

@stephen @lostplan @nathanchapman what's the move? if i dont hear anything in ~1 week, I'll go ahead and take care of it, with full dev/review attribution of course! trying to get this merged in the next month or so, my bad for accidentally leaving this hanging after the migration, so important! and it was ready! le sigh.

acao avatar Dec 12 '19 10:12 acao