astroidmail / astroid

A graphical threads-with-tags style, lightweight and fast, e-mail client for Notmuch
http://astroidmail.github.io
Other
613 stars 65 forks source link

Call set_size_request on pane children #755

Closed ibuclaw closed 4 months ago

ibuclaw commented 5 months ago

Otherwise the second child gets no width, and appears to be hidden when a thread is opened in pane view.

The number 300 is pretty arbitrary, it's taken as 1/4 of the default window width (set here).

Fixes #547

jorsn commented 5 months ago

Thanks, this works, and by default sets the pane size to half the window width. 300 seems pretty high to me, though, as it is the minimal width. Wouldn't 50 be enough?

Strangely enough, with only 20, the default width was smaller for me, maybe around the 300.

ibuclaw commented 4 months ago

I can't remember the effect of setting it to 50. Did initially toy with various sizes, I noticed that when too small, it didn't split the window evenly into two, rather split 2/3 and 1/3. Value I ended up with I think is a quarter the default width. On my screen any smaller and threads become mostly unreadable, but with a different font that would obviously change.