#70 Standardise icon sizes & reduce tab row min height

naomimmurphy commented 6 months ago

Not sure if you accept PRs, but here's a few changes I made locally with intentions aligned to #70, in case they are helpful:

Decrease minimum row height
Standardise margins & sizes for all icons to min row height
Avoid pin increasing line size
Move all size setting for icons to styles.css for ease of config & potential settings additions

Happy to make any changes if you have comments.

Particularly happy to revert to the existing size discrepancy between the buttons & the tab icon if you'd prefer (using --icon-l & --icon-s respectively) - on main the size of the parent div is not pulled through to the actual svg icons, so the pin is larger than expected (20x20 vs 18x18).

hdykokd commented 6 months ago

Thank you for your contribution! I will review the changes as soon as I have time.

hdykokd commented 6 months ago

Looks good. I think those are really great changes. I'm not very good with CSS styles, so I really appreciate you taking the time to work on this. Thank you.

hdykokd commented 6 months ago

I've released this as https://github.com/hdykokd/obsidian-vertical-tabs-view/releases/tag/2.4.6. @naomu Thank you!

hdykokd / obsidian-vertical-tabs-view

#70 Standardise icon sizes & reduce tab row min height #72