Dialog no longer alters the mainItem's visiblity, so this codepath failed.
It should have used Dialog's visiblity from the beginning anyway.
Details
Details
- Reviewers
lbeltrame - Group Reviewers
Plasma - Commits
- R119:67d8ac416ff6: Use Dialog's visibility directly rather than relying on the mainItem's
5.12?
Diff Detail
Diff Detail
- Repository
- R119 Plasma Desktop
- Lint
Automatic diff as part of commit; lint not applicable. - Unit
Automatic diff as part of commit; unit tests not applicable.
Comment Actions
That framework change is released tomorrow, right? Is it worth reverting?
Including this, that's at least the second regression it's caused, which makes me worry there's even more we haven't found.
It should have used Dialog's visiblity from the beginning anyway.
It should, so still ship this.