Bug 350157 - Widget & Apps icons added to *additional* (non-default) Plasma panel can not be relocated/reordered.
Summary: Widget & Apps icons added to *additional* (non-default) Plasma panel can not ...
Status: RESOLVED FIXED
Alias: None
Product: plasmashell
Classification: Plasma
Component: Panel (show other bugs)
Version: master
Platform: openSUSE Linux
: NOR major
Target Milestone: 1.0
Assignee: Plasma Bugs List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-07-12 22:34 UTC by Rob
Modified: 2015-07-13 20:28 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In: 5.4.0


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Rob 2015-07-12 22:34:56 UTC
I've installed

	rpm -qa | grep -i plasma-framework-5
		plasma-framework-5.13.0git.20150710T102506~1b8960c-1.1.x86_64
	plasmashell --version
		plasmashell 5.3.90
	lsb_release -rd
		Description:    openSUSE 13.2 (Harlequin) (x86_64)
		Release:        13.2

Create a new/empty Panel.
Reloacte the new panel to the top screen edge.
Add 2 or more Widgets or Apps to the top Panel.
Click to Edit the Panel
Attempt to relocate/reorder icons in the Panel.

Prior versions allowed the relocation, moving existing icons out of the way -- to the lefat & right -- of the selected/dragged icon's new target location.

Currently, nothing happens -- most of the time.

On occassion, simply retrying the same relocation, the icon will 'stick' in the new location, but no other icons move.  I.e., you end up with overlapped icons.

Then ~ 5-15secs later, everything will snap back to its original locations.

This does NOT occur on the original.default panel installed at the bottom of the screen; ONLY on the added panel.

Reproducible: Always
Comment 1 Kai Uwe Broulik 2015-07-13 08:43:46 UTC
Looks like a duplicate of Bug 349052 (each panel is a containment, like the Desktop, except that you can have multiple panels even on a single monitor setup). Could you perhaps try whether the bug is fixed with an up-to-date git master build of plasma-desktop?
Comment 2 Raymond Wooninck 2015-07-13 14:42:33 UTC
(In reply to Kai Uwe Broulik from comment #1)
> Looks like a duplicate of Bug 349052 (each panel is a containment, like the
> Desktop, except that you can have multiple panels even on a single monitor
> setup). Could you perhaps try whether the bug is fixed with an up-to-date
> git master build of plasma-desktop?

As indicated on IRC, the git commit is active on the system but does not resolve this particular bug. 

What I can reproduce is the following:   Add an empty panel to the desktop and add a number of widgets/plasmoids to it.  As long as you stay in the "unlock widgets" mode, then you can move these plasmoids around. However as soon as you lock them, they all jump back to the order they were initially added to the panel. Also what is weird is that if I move one plasmoid to another location, I can no longer move it anymore. Also when the mouse is hovering on it, the plasmoid name is not indicated.
Comment 3 Rob 2015-07-13 15:44:19 UTC
(In reply to Kai Uwe Broulik from comment #1)
> Looks like a duplicate of Bug 349052 (each panel is a containment, like the
> Desktop, except that you can have multiple panels even on a single monitor
> setup). Could you perhaps try whether the bug is fixed with an up-to-date
> git master build of plasma-desktop?

I upgraded to the distro's pkg:

  plasma5-desktop-5.3.91git~20150712T165117~87d5343-104.1.x86_64

I can't provide a changelog to verify that the fix is there; the distro says maintaining the changelog is 'too much work', so the latest commit log is from April 2015.

With that ^^ pkg in place though, the problem does NOT go away; same problem's seen as originally reported.
Comment 4 Kai Uwe Broulik 2015-07-13 17:43:48 UTC
Git commit 1217b9060f5490605832649116b29e94b448a3ec by Kai Uwe Broulik.
Committed on 13/07/2015 at 17:42.
Pushed by broulik into branch 'master'.

Only have one instance of the LayoutManager per containment

By removing the pragma library statement each QML file importing the LayoutManager
would gets its own independent JS context resulting in the main and ConfigOverlay
getting out of sync and causing all sorts of strange behavior.

Patch by notmart
FIXED-IN: 5.4.0

M  +5    -7    containments/panel/contents/ui/ConfigOverlay.qml
M  +1    -0    containments/panel/contents/ui/main.qml

http://commits.kde.org/plasma-desktop/1217b9060f5490605832649116b29e94b448a3ec
Comment 5 Rob 2015-07-13 20:28:55 UTC
(In reply to Kai Uwe Broulik from comment #4)
> Git commit 1217b9060f5490605832649116b29e94b448a3ec by Kai Uwe Broulik.

Confirming that upgrading to 

plasma5-desktop-5.3.91git~20150713T195602~4d726c4-107.1.x86_64

did the trick, thanks!

Responsiveness in the panel on icon relocation is VERY 'jerky' (searching bugs ...), and the Icon Settings dialog is still broken (bug filed), but I suspect these are two additional/separate issues from this.