-
Notifications
You must be signed in to change notification settings - Fork 25
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
new navigation pane for inspector #1142
Conversation
…tion and allows quicker navigation
Any opinion on this UI @fuhrmanator? (I tag you since you react on the Issue) |
Hello. Nice job. May be you could separate the associations (Access, inheritance, ...) from the Entities (Class, Method, ...) ? Another thing would be to also separate "technical entities (sourceAnchors, SourceLanguage) but there, I don't know how you could do it ? May be everything that inhrits from FamixSourceAnchor ? |
Ok, I'll look at these suggestions. Thanks for the proposal |
thanks for doing it ! |
This looks great! It's a huge improvement. Can't wait to try it on some TypeScript models. |
I integrated #1141 (comment) in this PR leading main navigation to looks like I propose to integrate this PR. It will add another navigation pane in second position so everybody can test both easily and then we can improve or change |
Be careful with the navigatino panes. There are alreadyt many. |
make test red but prepare for fix
For last version. I kill the navigation pane and the properties pane. In final this PR
But remove the sort of table of navigation properties |
This is a proposal for a new navigation pane for the Moose Inspector.
It is based solely on moose description for now.