Expose a 'filePicker' property on OxideQQuickWebView
Bug #1260008 reported by
Olivier Tilloy
This bug affects 3 people
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Oxide |
Fix Released
|
High
|
Olivier Tilloy |
Bug Description
For feature parity with QtWebKit, we need to expose a 'filePicker' property (or similar, in QtWebKit it’s 'experimental.
Related branches
lp:~osomon/oxide/filePicker2
- Chris Coulson: Approve
- Jamie Strandboge (community): Abstain
-
Diff: 1097 lines (+802/-2)25 files modifiedqt/core/browser/oxide_qt_file_picker.cc (+64/-0)
qt/core/browser/oxide_qt_file_picker.h (+49/-0)
qt/core/browser/oxide_qt_web_view.cc (+5/-0)
qt/core/browser/oxide_qt_web_view.h (+2/-0)
qt/core/core.gyp (+4/-0)
qt/core/glue/oxide_qt_file_picker_delegate.cc (+87/-0)
qt/core/glue/oxide_qt_file_picker_delegate.h (+66/-0)
qt/core/glue/oxide_qt_web_view_adapter.h (+2/-0)
qt/quick/CMakeLists.txt (+1/-0)
qt/quick/api/oxideqquickwebview.cc (+27/-1)
qt/quick/api/oxideqquickwebview_p.h (+6/-0)
qt/quick/api/oxideqquickwebview_p_p.h (+2/-1)
qt/quick/oxide_qquick_file_picker_delegate.cc (+152/-0)
qt/quick/oxide_qquick_file_picker_delegate.h (+57/-0)
qt/tests/qmltests/api/tst_WebView_filePicker.qml (+127/-0)
qt/tests/qmltests/api/tst_WebView_filePicker_data.html (+5/-0)
qt/tests/qmltests/api/tst_WebView_filePicker_data.txt (+1/-0)
qt/tests/qmltests/api/tst_WebView_filePicker_directory.html (+5/-0)
qt/tests/qmltests/api/tst_WebView_filePicker_multiple.html (+5/-0)
qt/tests/qmltests/api/tst_WebView_filePicker_single.html (+5/-0)
shared/browser/oxide_file_picker.cc (+47/-0)
shared/browser/oxide_file_picker.h (+56/-0)
shared/browser/oxide_web_view.cc (+19/-0)
shared/browser/oxide_web_view.h (+6/-0)
shared/shared.gyp (+2/-0)
Changed in oxide: | |
importance: | Undecided → High |
assignee: | nobody → Olivier Tilloy (osomon) |
Changed in oxide: | |
status: | New → In Progress |
tags: | added: desktop webapp-container |
tags: | added: webapps-blocker |
Changed in oxide: | |
status: | In Progress → Fix Released |
To post a comment you must log in.