Adds an unnecessary dependency to QtWidgets and forces some round-trips to the X server that we don't need.
Details
Details
- Reviewers
davidedmundson - Group Reviewers
Plasma - Commits
- R119:a8efa8653b09: Remove some uses on QDesktopWidget
Been using it for some days
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
concept ++
applets/pager/plugin/pagermodel.cpp | ||
---|---|---|
120 | we need this connection for screens that already exist when this code runs | |
containments/desktop/plugins/folder/foldermodel.cpp | ||
1640 | KNewFileMenu.cpp in the constructor without a transient parent we probably have focus issues? |
containments/desktop/plugins/folder/foldermodel.cpp | ||
---|---|---|
1640 | It doesn't show any issues that I can see. |
applets/pager/plugin/pagermodel.cpp | ||
---|---|---|
123 | Missing braces around single-line block and no const |