No need to load the keyboard view as soon as wayland starts. It's not a simple UI and it often will be unused.
Details
Details
- Reviewers
davidedmundson - Group Reviewers
KWin - Commits
- R108:5c39795b88d3: Load the keyboard on-demand
played with the keyboard, still worked
Diff Detail
Diff Detail
- Repository
- R108 KWin
- Branch
- master
- Lint
Lint Skipped Excuse: x - Unit
No Unit Test Coverage - Build Status
Buildable 13893 Build 13911: arc lint + arc unit
Comment Actions
The idea of loading on startup was to verify that the view works and only offer the sni and other interactions if the virtual keyboard is functional - see the early return in line 86. This is IMHO a must have feature as we cannot properly set a dependency on the virtual keyboard qml component. If we can check in another way, I'm all for this change.
Comment Actions
Given this patch still instantiates the engine and still creates the window object I think we'll get a simpler better patch putting a Loader inside main.qml which is bound to the window visibility.