-
Notifications
You must be signed in to change notification settings - Fork 19
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
Design improvements #17
Comments
What do you mean into the settings panel? The plugin settings? I feel that's quite an inconvenience if you want to quickly change algs. Maybe there is a better middle ground. And yes, I agree that the table header should be hidden if there are no notes |
@phlind You'll see I've slimmed down the AnalysisView a bit. Let me know your thoughts |
@phlind lots of new changes to the ui in 0.6.4! Let us know what you think |
Yes, I see many changes going on in the last two weeks, and I really like it, especially the scrolling mode. ❤️ Maybe one improvement that could be made is to make the font size the same as the file explorer. I just checked with the default theme and no snippets. Here the font looks bigger than the actual note text: Of course that's very easy to achive via CSS by every user, but maybe one could make it default? |
I would propose to make the plugin look less cluttered by moving the algorithm setting and the "zero" and "infinity" checkboxes into the settings panel. One has to really make the sidebar big, in order not to spread the settings across several lines.
Also, in the link predictions panel the table header should not be shown when there's no note

.
The text was updated successfully, but these errors were encountered: