Skip to content
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

Implement line highlight function on Editor #101

Merged
merged 6 commits into from
Apr 22, 2021

Conversation

dingge2016
Copy link
Sponsor Collaborator

@dingge2016 dingge2016 commented Mar 31, 2021

image

This PR adds support for trace graph -> editor interaction, #7

@laike9m
Copy link
Owner

laike9m commented Apr 2, 2021

I really appreciate this PR. Some general feedbacks:

cyberbrain-vsc/src/interactions.ts Outdated Show resolved Hide resolved
cyberbrain-vsc/src/interactions.ts Show resolved Hide resolved
cyberbrain-vsc/src/interactions.ts Outdated Show resolved Hide resolved
cyberbrain-vsc/src/interactions.ts Outdated Show resolved Hide resolved
cyberbrain-vsc/src/interactions.ts Outdated Show resolved Hide resolved
cyberbrain-vsc/src/rpc_server.ts Show resolved Hide resolved
cyberbrain-vsc/src/interactions.ts Outdated Show resolved Hide resolved
cyberbrain-vsc/src/rpc_server.ts Outdated Show resolved Hide resolved
cyberbrain-vsc/src/trace_graph.js Outdated Show resolved Hide resolved
… relative interaction

Implement a new interaction: When user hover a node on graph, highlight the node's line on the editor
1. add an overview ruler on the right side of the editor.
2. use different color to highlight the line in different themes.
@laike9m laike9m changed the title Implement line highlight function on Editor Implement line highlight function on Editor #7 Apr 22, 2021
@laike9m laike9m changed the title Implement line highlight function on Editor #7 Implement line highlight function on Editor Apr 22, 2021
@laike9m laike9m merged commit cfb91ac into laike9m:master Apr 22, 2021
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.

None yet

2 participants