Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
Adds linked editing for JSX tags #53284
Adds linked editing for JSX tags #53284
Changes from 30 commits
409d949
94c4b65
a5192e8
21aa9e7
398f673
2bafe8d
da0ddf1
9d7f32c
1fff99d
0b0541d
b70ee90
25393f1
bb98df0
6ba6321
92875ce
57ef9b8
06afbbd
f4ec7bd
803b292
34f5e40
03fa5bb
4faee99
6a1d3dd
c8ea559
fef7ab6
bb720e1
be34a1e
7f2f885
5164495
a3bdf2e
228e0ae
32e43c8
da4f1cd
db67aee
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm guessing LSP uses this name, but every other command type in the TS Server protocol is mostly camelCase.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's odd that the ranges are a subfield. Looking at existing precedent, there's
FileReferencesResponseBody
, so maybe:There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I had written the
LinkedEditingRanges
andLinkedEditingRangeResponse
this way, as this is how they were set up in the stubs and LSPThere was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would also prefer if it matched the style we had in tsserver and was named
Body
; LSP may define it one way but I don't think we should be inconsistent to match them.