edit mode action and dbus
ClosedPublic

Authored by mart on Sep 27 2019, 2:49 PM.

Details

Summary

edit mode menu action
edit mode dbus property for systemsettings
containmentlayoutmanager goes out of edit mode only when the whole
plasma app loses focus, so the panel controlled doesn't make it go out
of it

Test Plan

iaction shows in context menu, works
property exposed on dbus works

Diff Detail

Repository
R120 Plasma Workspace
Branch
mart/editMode
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 17175
Build 17193: arc lint + arc unit
mart created this revision.Sep 27 2019, 2:49 PM
Restricted Application added a project: Plasma. · View Herald TranscriptSep 27 2019, 2:49 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
mart requested review of this revision.Sep 27 2019, 2:49 PM
davidedmundson added inline comments.
components/containmentlayoutmanager/appletslayout.cpp
95–96

But we only evaluate this when this window changes - doesn't that mean we could when any other plasma window had focus and then loses it?

I wonder if the widget resize handles should become visible all the time when in edit mode. Otherwise nothing visually changes with the widgets on the desktop, and it might not be obvious enough that you can change them.

mart added a comment.Sep 30 2019, 10:45 AM

I wonder if the widget resize handles should become visible all the time when in edit mode. Otherwise nothing visually changes with the widgets on the desktop, and it might not be obvious enough that you can change them.

I tought about it too, tough maybe if everyhting is visible (right now the resize handles and the bigger applet handle are linked in visibility) wouldn't look very messy and cluttered?

components/containmentlayoutmanager/appletslayout.cpp
95–96

right.. it wouldn't notice if the edit mode was initiated when its window not in focus, i-.e. dbus from systemsettings case.
that may probably mean the logic should be completely removed from here and put in shellcorona i guess?

mart updated this revision to Diff 67071.Sep 30 2019, 2:54 PM
  • move logic of out of editmode in shellcorona
mart marked an inline comment as done.Sep 30 2019, 2:55 PM
mart added inline comments.
components/containmentlayoutmanager/appletslayout.cpp
95–96

different approach in current version

In D24264#539730, @mart wrote:

I thought about it too, tough maybe if everything is visible (right now the resize handles and the bigger applet handle are linked in visibility) wouldn't look very messy and cluttered?

Maybe, but this is an explicit edit mode, so it might make sense to have the editing controls more obvious?

mart added a comment.Oct 1 2019, 9:40 AM

As it can be seen there, we can have situations like applets going over the handle of another one or resize handles perfectly overlapping one another, and that can't be avoied technically, it looks simply broken
(and no, i don't want to move everything inside the applet area because they can become very small, as small as a single icon, in which would make the controls unusable, at least in touch mode, so i consider the design of the handles pretty much final)
there may be added something visible that's shown which are not the handles, not sure what

GB_2 added a subscriber: GB_2.Oct 1 2019, 9:48 AM

Just showing the handles when you hover over a widget (or tap on one on touch) is fine IMO.

In D24264#540228, @GB_2 wrote:

Just showing the handles when you hover over a widget (or tap on one on touch) is fine IMO.

Yeah that's probably a reasonable compromise.

mart added a comment.Oct 1 2019, 2:35 PM
In D24264#540228, @GB_2 wrote:

Just showing the handles when you hover over a widget (or tap on one on touch) is fine IMO.

Yeah that's probably a reasonable compromise.

there should be code almost ready for that, shouldn't be hard, i'll try

mart updated this revision to Diff 67166.Oct 2 2019, 8:11 AM
  • edit mode on mouse over when containment in edit mode
mart added a comment.Oct 2 2019, 8:12 AM

edit mode on mouse over

mart updated this revision to Diff 67185.EditedOct 2 2019, 1:16 PM

Introduce eventManagerToFilter to make long press work on folderview containment too

ngraham accepted this revision as: VDG.Oct 2 2019, 2:52 PM
ngraham added a reviewer: VDG.
mart updated this revision to Diff 67518.Oct 8 2019, 5:20 PM
  • don't trigger edit mode with right button
mart updated this revision to Diff 67564.Oct 9 2019, 3:11 PM
  • show desktop when in edit mode
mart updated this revision to Diff 67707.Oct 11 2019, 2:23 PM
  • only close with proper touch events
ngraham accepted this revision.Oct 11 2019, 3:26 PM
This revision is now accepted and ready to land.Oct 11 2019, 3:26 PM
GB_2 added a comment.EditedOct 12 2019, 1:47 PM

It still leaves the edit mode when closing a panel controller toolbox which isn't consistent with the widget explorer and it has a few side effects such as not being able to edit three panels or not being able to add widgets from the panel controller toolbox and keeping the panel controller toolbox open.

mart added a comment.Oct 14 2019, 1:34 PM
In D24264#546001, @GB_2 wrote:

It still leaves the edit mode when closing a panel controller toolbox which isn't consistent with the widget explorer and it has a few side effects such as not being able to edit three panels or not being able to add widgets from the panel controller toolbox and keeping the panel controller toolbox open.

fixed in latest version of D24265

GB_2 accepted this revision as: GB_2.Oct 18 2019, 8:38 AM
davidedmundson accepted this revision.Oct 18 2019, 11:27 AM
davidedmundson added inline comments.
components/containmentlayoutmanager/appletslayout.cpp
517

the first clause seems redundant

543

Please comment what this is doing.

want with normal mouse the edit mode not being dismissed when clicking on empty areas but to do it on touch

shell/shellcorona.cpp
220

that's clever

mart updated this revision to Diff 68234.Oct 18 2019, 12:40 PM
  • explanation++
  • remove redundant check
mart marked 3 inline comments as done.Oct 18 2019, 12:43 PM
This revision was automatically updated to reflect the committed changes.