joplin-plugin-knowledge-graph icon indicating copy to clipboard operation
joplin-plugin-knowledge-graph copied to clipboard

Add ability to hide controls

Open shbach opened this issue 3 years ago • 8 comments

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 avatar Dec 23 '21 21:12 shbach

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

shbach avatar Dec 27 '21 21:12 shbach

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...

agerardin avatar Dec 27 '21 22:12 agerardin

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.

shbach avatar Dec 27 '21 23:12 shbach

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.

agerardin avatar Dec 31 '21 04:12 agerardin

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 avatar Dec 31 '21 06:12 shbach

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).

shbach avatar Jan 05 '22 21:01 shbach

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

agerardin avatar Jan 14 '22 18:01 agerardin

New option works great!

shbach avatar Jan 16 '22 20:01 shbach