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-11e6-9d87-0cc152fd681d.png)
 | ![gctb_3 
18_2](https://cloud.githubusercontent.com/assets/793526/14656124/0a3eb754-0686-11e6-8095-c721dd92c756.png)
GTK 3.20           | ![gctb_3 
20_1](https://cloud.githubusercontent.com/assets/793526/14656130/13cf17b4-0686-11e6-9fcc-e12bf484954b.png)
 | ![gctb_3 
20_2](https://cloud.githubusercontent.com/assets/793526/14656135/182054d6-0686-11e6-8399-765eacc8ec8c.png)
GTK 3.20 + this PR | ![gctb_3 20 
pr_1](https://cloud.githubusercontent.com/assets/793526/14656137/1ce74a56-0686-11e6-9950-28cbb4b016e3.png)
 | ![gctb_3 20 
pr_2](https://cloud.githubusercontent.com/assets/793526/14656141/206bac08-0686-11e6-8164-d7cbbc456b6a.png)

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

Reply via email to