Create containment on specified screen
Needs ReviewPublic

Authored by hoffmannrobert on Feb 6 2018, 2:57 PM.

Details

Reviewers
None
Group Reviewers
Plasma
Summary

Add methods used by Plasma::Containment *ShellCorona::addPanel(const QString &plugin) (see plasma-workspace: https://phabricator.kde.org/D10344 ):
Containment *createContainmentForScreen(int screenNum, const QString &name, const QVariantList &args = QVariantList());
Containment *addContainmentForScreen(int screenNum, const QString &name, const QVariantList &args, uint id, bool delayedInit = false);
void Containment::setLastScreen(int screen);

These are needed to create panels on the current screen in multi-monitor environments, so panels can be created on all desired screens.

Currently it is only possible to have panels on one screen.

Test Plan

More than one monitor connected: Right-click on empty desktop, "Add Panel", choose any ("Empty", "Standard",...). Panel appears on monitor where you opened the context menu.

Diff Detail

Repository
R242 Plasma Framework (Library)
Branch
master
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 1167
Build 1180: arc lint + arc unit
hoffmannrobert created this revision.Feb 6 2018, 2:57 PM
Restricted Application added a project: Frameworks. · View Herald TranscriptFeb 6 2018, 2:57 PM
Restricted Application added a subscriber: Frameworks. · View Herald Transcript
hoffmannrobert requested review of this revision.Feb 6 2018, 2:57 PM
hoffmannrobert edited the summary of this revision. (Show Details)Feb 6 2018, 2:59 PM
hoffmannrobert edited the test plan for this revision. (Show Details)Feb 6 2018, 3:09 PM
hoffmannrobert edited the test plan for this revision. (Show Details)
  • Remove unnecessary Containment::setLastScreen()
Restricted Application edited subscribers, added: kde-frameworks-devel; removed: Frameworks. · View Herald TranscriptJul 25 2018, 11:40 AM