When using Dialog or QueryDialog, it should be possible to set hideOnWindowDeactivate flag.
CCBUG: 401016
Details
Details
- Reviewers
davidedmundson - Group Reviewers
Plasma Frameworks - Commits
- R242:96b7fc21f5b5: Add hideOnWindowDeactivate to PlasmaComponents.Dialog
Diff Detail
Diff Detail
- Repository
- R242 Plasma Framework (Library)
- Lint
Automatic diff as part of commit; lint not applicable. - Unit
Automatic diff as part of commit; unit tests not applicable.
Comment Actions
This is a first step to solve 401016:
"Touchpad plasmoid [confirmation dialog] remains open when losing focus, I think that it is inconsistent since other elements of tray close themselves after switching away."
AFAIK PlasmaComponents 2 are deprecated. Is there newer QueryDialog replacement?
Comment Actions
Should I proceed? This component has serious issues anyway, maybe there is no point in fixing this?
For example:
There is Loader that load always the same QML because... "if (true || ....)" - line 241. Even if this is obsolete, cleanup would be good idea.
Comment Actions
maybe there is no point in fixing this?
PlasmaComponents are somewhat deprectaed in favour of Plasma Components 3 (which have to match QtQuickControls2 API)
So maybe not, from what I can see the original code could use PlasmaCore.Dialog directly. Or the inbuilt way plasma does popups.
On the other hand, there's no harm to this patch either.