davidedmundson added a comment.
I love use of layouts for everything, so in general ++
INLINE COMMENTS
> CompactApplet.qml:34
> subText: plasmoid.toolTipSubText
> - location: if (plasmoid.parent && plasmoid.parent.parent.objectName ===
> "hiddenTasksColumn" && plasmoid.location !== PlasmaCore.Types.LeftEdge) {
> - return PlasmaCore.Types.RightEdge;
When you say it did nothing, You'd need to test every hidden applet whilst
having the panel on the left.
See D1253 <https://phabricator.kde.org/D1253>
If it ain't broke...
> PlasmoidItem.qml:45
> if (applet && mouse.button === Qt.LeftButton) {
> - applet.expanded = true;
> + applet.expanded = !applet.expanded;
> }
Huh? how did this work before?
> main.qml:66
>
> + // Shouldn't it be part of Qt?
> + function findParentNamed(object, objectName) {
Not really.
Using objectNames is a bit of an anti pattern, especially when QML has so the
built-in component scope hierachy.
We use it in the system tray already, and it's arguably no worse than the
existing applet.parent.parent.
So fine here, but only because the system is mad.
REPOSITORY
R120 Plasma Workspace
REVISION DETAIL
https://phabricator.kde.org/D19745
To: ratijastk, #vdg, #plasma, broulik, mart, hein
Cc: davidedmundson, ngraham, ndavis, anthonyfieroni, plasma-devel, jraleigh,
GB_2, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts,
sebas, apol, mart
