Skip to content
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

GTK3 TODO #112

Open
bp0 opened this issue Jul 25, 2017 · 7 comments
Open

GTK3 TODO #112

bp0 opened this issue Jul 25, 2017 · 7 comments

Comments

@bp0
Copy link
Collaborator

bp0 commented Jul 25, 2017

GTK3 support was added in #114, but it still has a few issues.

Status

  • Hardinfo builds and runs without crashing on GTK3.
  • GTK2 continues to work as well as it ever did.
  • There are many deprecation warnings during build.
  • loadgraph is replaced by Christian Hergert's uber-graph widget for GTK3 hardinfo
  • guibench is disabled for gtk3.
  • look for TODO:GTK3 in the code to find places that need work.
grep -nr --include='*.[hc]' "TODO:GTK3" *
shell/report.c:916:/* TODO:GTK3
shell/shell.c:366:    /* TODO:GTK3 css-based style */
shell/shell.c:855:    /* TODO:GTK3 */
shell/shell.c:888:    /* TODO:GTK3 */
shell/shell.c:1616:     /* TODO:GTK3 gtk_alignment_new(), etc is deprecated from 3.14 */
shell/shell.c:1780:    /* TODO:GTK3 */

Testing

mkdir build && cd build
cmake -DHARDINFO_GTK3=1 ..
[etc.]

Things to look at

  • modules/computer/display.c: gdk_screen_* all deprecated in 3.22.
@bp0 bp0 mentioned this issue Jul 25, 2017
@lpereira
Copy link
Owner

As far as the load graph goes, it might be a good idea to use this instead of just porting the current code to Cairo: https://github.com/chergert/uber-graph

It supports multiple graphs, looks nicer, etc.

@bp0
Copy link
Collaborator Author

bp0 commented Jul 25, 2017

That may be better, but would it be easier?

@lpereira
Copy link
Owner

It's not going to be easier, no. For the porting work it might be better to stick with the conversion from GDK drawing primitives to Cairo. Maybe afterwards move it to uber-graph.

@bp0
Copy link
Collaborator Author

bp0 commented Jul 25, 2017

Right, I just want to see it build for gtk3. I still have the link problem that I mentioned at #59 (comment)

@bp0
Copy link
Collaborator Author

bp0 commented Jul 25, 2017

GTK3 Hardinfo (in XFCE)
hardinfo-gtk3
Broken:
hardinfo-gtk3-ohno

@bp0
Copy link
Collaborator Author

bp0 commented Jul 26, 2017

@lpereira
I was wrong. Just using uber-graph was significantly easier. Check out #114.

@bp0
Copy link
Collaborator Author

bp0 commented Jul 26, 2017

GTK3 and uber-graph:
temp-spike

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants