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

feat: Option open node with shift works also in Local mode. #141

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

hayio
Copy link

@hayio hayio commented Jan 22, 2024

At this moment Option open with shift works only in Workspace mode.
It's confusing, especially that it this Option is under Advanced section, and not under Workspace mode.
I've added added same code for th local mode as I think it could be helpful also there to traverse a graph without switching to every file.

@HEmile
Copy link
Owner

HEmile commented Apr 12, 2024

Sorry for not replying earlier, I missed this (January was hectic!)

Do I understand correctly that if you click on a node in local mode, this will move towards that node, and if you shift-click, it will expand around it and move?

@hayio
Copy link
Author

hayio commented Apr 18, 2024

:)
Let's say we have A->B->C.
In local mode, the initial situation is that you have opened tab with a text view of node A and second tab with its juggl local graph A->B.
Then if you click on B, only the tab with local juggl graph will update to show B->C, leaving the tab with text view of A untouched.
Then, if you click with SHIFT on C then both tabs (text and juggl) will switch to display C - same behaviour as when you have this shift option turned off.
The juggl view bahaviour (traversing and expansions) is no different. Only the text view differs.

This way you could wander the graph without leaving your current node in the editor. The navigation history would also be shorter, boiled down to only shift-selected nodes.

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.

2 participants