Skip to content

Avoid recursive selection changes in ast viewer#668

Merged
aeisenberg merged 1 commit intogithub:mainfrom
aeisenberg:aeisenberg/astviewer-recursive
Nov 4, 2020
Merged

Avoid recursive selection changes in ast viewer#668
aeisenberg merged 1 commit intogithub:mainfrom
aeisenberg:aeisenberg/astviewer-recursive

Conversation

@aeisenberg
Copy link
Copy Markdown
Contributor

@aeisenberg aeisenberg commented Nov 3, 2020

This will prevent selections jumping around when an ast entry is
selected and its child has the same source location as the current
selection.

Fixes #656.

Checklist

  • [n/a] CHANGELOG.md has been updated to incorporate all user visible changes made by this pull request.
  • Issues have been created for any UI or other user-facing changes made by this pull request.
  • [n/a] @github/docs-content-dsp has been cc'd in all issues for UI or other user-facing changes made by this pull request.

This will prevent selections jumping around when an ast entry is
selected and its child has the same source location as the current
selection.
@aeisenberg aeisenberg added the Complexity: Low A good task for newcomers to learn, or experienced team members to complete quickly. label Nov 3, 2020
@aeisenberg
Copy link
Copy Markdown
Contributor Author

Turned out to be a lot simpler than I expected. There's an API for this.

@aeisenberg aeisenberg merged commit 2953c15 into github:main Nov 4, 2020
@aeisenberg aeisenberg deleted the aeisenberg/astviewer-recursive branch November 24, 2020 22:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Complexity: Low A good task for newcomers to learn, or experienced team members to complete quickly.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

AST Viewer selection bounces around

3 participants