-
Notifications
You must be signed in to change notification settings - Fork 252
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
Adding QWebEngineView #273
base: master
Are you sure you want to change the base?
Conversation
I committed this with the wrong author email. I'll update and re-pull-request. |
It looks like github managed my edit: It looks like Travis is failing because I blindly setup the remap from I'll see if I can get this working with |
Those modules won't be available on all bindings, in accordance with the VFX Platform specs, as you can see from the failing tests. I would recommend you look into QtSiteConfig.py to add these on your end. |
Continuing from the Maya Python mailing list. @krets Thanks for the pull request. I haven't used these web widgets myself, but my impression is that they are too different to be treated as one and the same; one wrapping WebKit and the other Chromium. For completeness, the members exposed through Qt.py are those that are identical across all bindings, such that you can write an application in any binding and know for sure it will run without problems in all other bindings without changes to the original application. If you are confident that QWebView qualifies, then I see no problem including it in Qt.py. What I'd need is tests that exercise the most important functionality of QWebView. Functionality that isn't portable should be made an example of as Caveats. Let me know if you have any questions! |
@krets in case QtC decided to move members around between Qt4 and Qt5 in regards to QtWebEngineWidgets, you need to figure out (seems like you already did) how these were moved/renamed and then we'll have to include this here: https://github.com/mottosso/Qt.py/blob/master/membership.py#L62 The So the idea is that we should copy the So in this case, you should make sure that you can see the following in the generated
Feel free to rename the |
Edited my previous post for clarity.. |
This gives me enough to port my existing
QWebView
widgets intoQWebEngineView
.