search

agerardin / joplin-plugin-knowledge-graph

Notes as nodes. Explore your Joplin knowledge graph.
MIT License
93 stars 5 forks source link

Add ability to hide controls #6

Open shbach opened 2 years ago

shbach commented 2 years ago

A lot of the options currently displayed on the right sidebar could potentially be simplified and toggled/hidden in some way to give more space for the graph itself. Always displaying these options adds clutter when the graph itself is already busy and takes up too much space on smaller screens.

shbach commented 2 years ago

On smaller screens, the options won't all fit and can't be accessed: image

agerardin commented 2 years ago

CSS heaven! May I ask what size of screen and what resolution you are using? I do need to add small screen/low res to my test workflow and brush up on my CSS... Also looking at collapsing the whole menu without adding bulky dependencies...

shbach commented 2 years ago

1920x1080 on a 14 inch display and also a 32 inch display. I wouldn't consider the latter a small screen but 1080p is still too low resolution for to show everything.

agerardin commented 2 years ago

I had to work hard on my CSS skills but I finally implemented a collapsible menu you can show and hide and made everything more tidy and better looking. Thanks for the great suggestion! It was indeed though to use the plugin on lower res displays. Only tested on 4K before so I had missed it.

shbach commented 2 years ago

Other than addressing the horizontal scroll bar which I mentioned, I think this issue can be closed now.

Edit to add more info on this: The horizontal scrollbar isn't really useful to show because if there was a scenario in which it could be used, the graph wouldn't be able to show anyways. Also it takes up space and could make the difference between the vertical scrollbar being needed or not. image

shbach commented 2 years ago

One more thing which would improve the user experience is for the plugin to remember if the controls are hidden/expanded upon restart. Right now whenever I restart Joplin the controls are always expanded regardless of what I previously set.

Alternatively, just hide the controls by default (maybe have an option under settings to change this behavior for people who don't want this).

agerardin commented 2 years ago

with release 1.1.0 there is an option to remember to show the control panel on startup or not.

shbach commented 2 years ago

New option works great!