Bug 365453

Summary:	The extended output should NOT be isFolder for extended mode
Product:	[Plasma] plasmashell	Reporter:	Leslie Zhai <zhaixiang>
Component:	generic-multiscreen	Assignee:	Aleix Pol <aleixpol>
Status:	RESOLVED FIXED
Severity:	normal	CC:	plasma-bugs-null, sebas
Priority:	NOR
Version First Reported In:	5.7.0
Target Milestone:	1.0
Platform:	Other
OS:	Linux
Latest Commit:		Version Fixed/Implemented In:
Sentry Crash Report:

Description Leslie Zhai 2016-07-12 06:17:10 UTC

It is a tiny issue: when plugin a new output (monitor) and in extended mode, the primary screen's desktopcontainment is FolderView by default, but the extended screen is FolderView too! it is better to be DesktopView.

So it needs to add primary screen check and current mode for property bool isFolder: (plasmoid.pluginName == "org.kde.plasma.folder")

Reproducible: Always

Comment 1 Sebastian Kügler 2016-08-08 11:36:31 UTC

I don't get it. plasmashell has never created a folderview containment for me by default.

What's your distro? This seems like the distro has changed the default to be folderview.

Comment 2 Leslie Zhai 2016-08-09 01:13:40 UTC

Hi Sebastian,

ArchLinux, for 5.7.0 it is able to reproduce when pressed Qt::Key_Display or Qt::MetaModifier + Qt::Key_P several times, even the primary output will be DesktopView wrongly, I will test the master branch.

Regards,
Leslie Zhai

Comment 3 Leslie Zhai 2016-08-11 02:57:26 UTC

Fixed! it can not be reproduced for libkscreen and kscreen master ;-)

the extended screen is DesktopView correctly!