Set margin_top on the revealer bars in addition to margin_bottom

astroidmail / astroid

A graphical threads-with-tags style, lightweight and fast, e-mail client for Notmuch

http://astroidmail.github.io

Other

606 stars 65 forks source link

Closed ibuclaw closed 4 weeks ago

ibuclaw commented 1 month ago

Sets equal margins on both top and bottom, which (to me) has a better aesthetic look to it.
I noticed that there was a typo in the code which meant the bulk update ("multi") revealer never got a margin set.

Before this PR:

After this PR:

ibuclaw commented 1 month ago

After a little toying around, I've convinced myself that a slightly smaller margin for the yes/no buttons looks better.

Updated "After" screenshot:

jorsn commented 4 weeks ago

Thanks!