New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
treebrowser: fixed deprecated gtk calls for gtk3 #864
Conversation
@medovina What do you think about it? |
Apologies for the delay in responding - the last few weeks have been super busy. I tried building with these changes. It's nice to see that all the build warnings are gone! On my machine (running Ubuntu 19.04), without your changes lines in the TreeView are gray, not red. With your changes, they are black. I personally think the gray looks better - the black lines are too distracting. What operating system / distribution are you running? That could have an effect on the TreeView line colors. If possible, I think it would be better to use a color from the underlying GTK theme rather than hard-coding black. |
Never mind - we are all volunteers.
I am using Ubuntu 18.04.2 and a gnome desktop. I lately implemented the option to show tree lines for the workbench plugin's sidebar and there I also see red lines. So most likely you are right - it seems to be a local problem not a general one. I will remove the CSS part which sets the border color. |
6320a5a
to
70c6aa0
Compare
Done. The tree-line color (border-color) is now not explicitly set (so it is red again on my system). |
OK - I built with your latest changes. They look fine to me. Do you know why the Travis bulid above failed? That might be worth investigating. |
There seemed to be a problem with devhelp on the gtk3 build only:
I got the impression that this sometimes happens and sometimes not. But it's not related to the treebrowser changes. |
I saw this and I don't see any issue with the Devhelp make file, it looks pretty normal. Also some other PRs are not failing which is weird. @b4n any ideas? I haven't tried to do a |
See #878 |
@medovina: do you request any more changes? I would like to merge this soon. |
Fine with me to merge. |
Apart from fixing a load of deprecation warnings on gtk3 this also fixes two gtk3 related problems in the treebrowser plugin:
The gtk2 behavior is unchanged and there are also no functional changes.
If this get's merged PR #279 would become needless and could be closed un-merged.