Well, it's better but still not really nice.
| normal | hover -------------------|--------|------ GTK 3.18 | ![gctb_3 18_1](https://cloud.githubusercontent.com/assets/793526/14656121/06aedf24-0686-11e...) | ![gctb_3 18_2](https://cloud.githubusercontent.com/assets/793526/14656124/0a3eb754-0686-11e...) GTK 3.20 | ![gctb_3 20_1](https://cloud.githubusercontent.com/assets/793526/14656130/13cf17b4-0686-11e...) | ![gctb_3 20_2](https://cloud.githubusercontent.com/assets/793526/14656135/182054d6-0686-11e...) GTK 3.20 + this PR | ![gctb_3 20 pr_1](https://cloud.githubusercontent.com/assets/793526/14656137/1ce74a56-0686-11e...) | ![gctb_3 20 pr_2](https://cloud.githubusercontent.com/assets/793526/14656141/206bac08-0686-11e...)
I don't really think it's great as is. Both 3.20 versions get the button too close hiding a part of the label, and none make the button really any kind of smaller. With Adwaita at least it doesn't change the tab's effective height, which is good, but we'd have to see with other themes if we really care. And the version in this PR doesn't seem to get the button really smaller, but removes the border when hovering, making the visual clue worse.
I guess the code here is fine, but the CSS needs a little more love.
I can confirm that this change repairs the font on the sci autocomplete popup (without this it uses a proportional font).
That doesn't make sense as nothing in our CSS touches that. Also, it doesn't fix it for me.
--- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/994#issuecomment-212138827