-
Notifications
You must be signed in to change notification settings - Fork 380
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
pyqt5 #205
base: dev
Are you sure you want to change the base?
pyqt5 #205
Conversation
…n't use ghost in multithreaded app when using "main" app.
I have tested this one and it does work without impairing the old interface - maybe worth adding to the mainstream? |
It will be great! But probably there is still some issues with processes and/or threads, which we don't know about right now. |
Is there anything we could do to help with merging this? @bollwyvl could do with having this merged for some work we're doing and we could plan to put aside time to help get the PR merged. |
A bit of background: we're generating PDF of Jupyter Notebooks with ghost, and it's pretty good. The issue that's tripping me up for a class of generation is in connecting to pages that use WebSockets, and a number of special flags, etc. need to be set, etc... so it might be more of an inconvenience than a requirement, as I haven't full explored that space. However, as I continue testing, I imagine I'll encounter limitations of QtWebKit, whether javascript, CSS or just craziness, as the pages I am rendering can indeed get pretty crazy. Having done some more research on the matter: apparently, QWebKit hasn't been updated for some time. Thus, getting ghost to work with it's latest version would be nice, if easy, as we would get to the last state of improvement webkit is likely to get. A high-value goal for emulating desktop browsers even more closely would be the Qt5-only QtWebEngine, which is basically chromium. Among other things, you can connect to it from the chrome dev tools which is magic. QtWebEngine is not supported by pyqt, but there is at least one example of using it via round-about methods... and may not expose all of the APIs ghost would need. pyside2 does have support for QtWebEngine, but it's not ready yet. So if really significant work is required to get ghost to work with qt5, it may be worth waiting until one of the upstreams supports QtWebEngine to invest significant work. Of the two possible upstreams, pyside is the one that would actually accept effort, as I couldn't even find the code for pyqt in non-dump-from. |
Nice idea to have pyqt5, I think. Probably I missed something with pyqt4 <-> pyqt5 difference, as they are pretty similar.