D8566: Add API to retrieve the screen id for a screen name
This revision was automatically updated to reflect the committed changes. Closed by commit R242:eab03787350a: Add signals indicating if a screen was added or removed (authored by amantia). REPOSITORY R242 Plasma Framework (Library) CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D8566?vs=21682=22339 REVISION DETAIL https://phabricator.kde.org/D8566 AFFECTED FILES src/plasma/corona.h To: amantia, #plasma, ervin, dvratil, mlaurent, hein, davidedmundson Cc: broulik, plasma-devel, dhaumann, apol, #frameworks, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, mart
D8566: Add API to retrieve the screen id for a screen name
amantia added a dependent revision: D8493: Make Folder View screen aware. REPOSITORY R242 Plasma Framework (Library) REVISION DETAIL https://phabricator.kde.org/D8566 To: amantia, #plasma, ervin, dvratil, mlaurent, hein, davidedmundson Cc: broulik, plasma-devel, dhaumann, apol, #frameworks, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, mart
D8566: Add API to retrieve the screen id for a screen name
amantia updated this revision to Diff 21682. amantia added a comment. Add signals instead of a new virtual method (that is not BIC). This also makes us not rely on screen name, but only on plasma-internal screen id. REPOSITORY R242 Plasma Framework (Library) CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D8566?vs=21679=21682 BRANCH master REVISION DETAIL https://phabricator.kde.org/D8566 AFFECTED FILES src/plasma/corona.h To: amantia, #plasma, ervin, dvratil, mlaurent, hein Cc: broulik, plasma-devel, dhaumann, apol, #frameworks, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, mart
D8566: Add API to retrieve the screen id for a screen name
amantia added a comment. @broulik Very good point, luckily I reworked it. REPOSITORY R242 Plasma Framework (Library) REVISION DETAIL https://phabricator.kde.org/D8566 To: amantia, #plasma, ervin, dvratil, mlaurent, hein Cc: broulik, plasma-devel, dhaumann, apol, #frameworks, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, mart
D8566: Add API to retrieve the screen id for a screen name
amantia updated this revision to Diff 21679. amantia added a comment. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. Fix API docs text REPOSITORY R242 Plasma Framework (Library) CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D8566?vs=21617=21679 BRANCH master REVISION DETAIL https://phabricator.kde.org/D8566 AFFECTED FILES src/plasma/corona.cpp src/plasma/corona.h To: amantia, #plasma, ervin, dvratil, mlaurent, hein Cc: plasma-devel, dhaumann, apol, #frameworks, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, mart
D8566: Add API to retrieve the screen id for a screen name
amantia edited the summary of this revision. amantia added reviewers: Plasma, ervin, dvratil, mlaurent, hein. REPOSITORY R242 Plasma Framework (Library) REVISION DETAIL https://phabricator.kde.org/D8566 To: amantia, #plasma, ervin, dvratil, mlaurent, hein Cc: apol, #frameworks
D8566: Add API to retrieve the screen id for a screen name
amantia added a comment. I will add, please be patient as there are 3 related commits :) REPOSITORY R242 Plasma Framework (Library) REVISION DETAIL https://phabricator.kde.org/D8566 To: amantia Cc: apol, #frameworks
D8566: Add API to retrieve the screen id for a screen name
amantia created this revision. Restricted Application added a project: Frameworks. Restricted Application added a subscriber: Frameworks. REPOSITORY R242 Plasma Framework (Library) BRANCH master REVISION DETAIL https://phabricator.kde.org/D8566 AFFECTED FILES src/plasma/corona.cpp src/plasma/corona.h To: amantia Cc: #frameworks