Details
-
Task
-
Resolution: Unresolved
-
P2: Important
-
None
-
None
-
None
-
a82862064 (dev), 92b032324 (dev), 2ab07b455 (dev), 8f7fc8e9a (dev), b2a433c33 (dev), 82cf2f08d (dev), bcbe7e233 (dev), c57dcfdb9 (dev), 75155ce5c (dev), c8dd0c877 (dev)
Description
QWasmWindow today has for the most part the behavior we want, but has gone through several transitons (e.g. canvas per screen -> canvas per window) which has left the code with some need for cleanup and refactoring.
Tasks include:
- Facilitate adding support for QWindow::fromWinId() (which native element is "the window"?)
- Minimize usage of html elements
- Add "window manager" elements lazily (i.e. not for "fullscreen" windows)
- Move "contentEditable" away from the canvas
- Move all client event handlers to the window element.
Current screen/window html element tree:
Target screen/window html element tree:
Attachments
Gerrit Reviews
| For Gerrit Dashboard: QTBUG-128732 | ||||||
|---|---|---|---|---|---|---|
| # | Subject | Branch | Project | Status | CR | V |
| 589595,3 | wasm: rename canvasResizeObeserver -> resizeObserver | dev | qt/qtbase | Status: NEW | +2 | 0 |
| 588784,3 | wasm: add window management manual test | dev | qt/qtbase | Status: MERGED | +2 | 0 |
| 589586,3 | wasm: rename canvas CSS class to qt-window-canvas | dev | qt/qtbase | Status: MERGED | +2 | 0 |
| 589587,3 | wasm: don't set contentEditable on the canvas | dev | qt/qtbase | Status: MERGED | +2 | 0 |
| 589588,4 | wasm: remove superfluous m_canvasContainer | dev | qt/qtbase | Status: MERGED | +2 | 0 |
| 589589,4 | wasm: move input event handlers to windowContents | dev | qt/qtbase | Status: MERGED | +2 | 0 |
| 589590,4 | wasm: remove QWasmWindow::window() and m_window | dev | qt/qtbase | Status: MERGED | +2 | 0 |
| 589591,5 | wasm: rename qt-window -> qt-decorated-window | dev | qt/qtbase | Status: MERGED | +2 | 0 |
| 589592,4 | wasm: rename m_windowContents -> m_window | dev | qt/qtbase | Status: MERGED | +2 | 0 |
| 589593,4 | wasm: remove QWasmScreen::eventTargetId() | dev | qt/qtbase | Status: MERGED | +2 | 0 |
| 589594,4 | wasm: rename updateQScreenAndCanvasRenderSize() | dev | qt/qtbase | Status: MERGED | +2 | 0 |