-
Notifications
You must be signed in to change notification settings - Fork 148
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
wxWidgets and Qt support #9
Comments
Since #35 has been merged, all the GUI-related code is solely in I've been trying to get my head around how to do this sensibly, and I can see two different ways of doing it:
The first way would almost certainly be easier, but managing separate packages for each GUI might be difficult. The second way seems maybe cleaner, but I'm not even sure where to begin on that. |
I tried my hand at rebasing the Qt fork in the original question (with Python 3 and PyQt5) and the split is just short of being sufficient to abstract away the UI: in the Qt fork, To avoid this problem, |
@Morwenn, do you have your fork somewhere? It seems that you made all the work and you're just a pull request or two away from integrating it. |
It's probably somewhere on my work computer. We didn't use the library in the end, but LGPL says that I should share the improvements anyway. I will try to find it again once I'm back from holidays. If I'm not mistaken I had only kept the code up to the widget, and not up to the window level. |
@jrfonseca, we've been using Xdot for several years now with a wxWidgets-based fork and, more recently, a Qt-based fork. It'd be cool to unify them into a single library, and combine the common code.
Check them out here: https://github.com/jbohren/xdot/tree/indigo-devel/src/xdot
The text was updated successfully, but these errors were encountered: