<table><tr><td style="">sitter requested changes to this revision.<br />sitter added a comment.<br />This revision now requires changes to proceed.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D12380">View Revision</a></tr></table><br /><div><div><p>quick dump from irc</p>

<div class="remarkup-code-block" style="margin: 12px 0;" data-code-lang="text" data-sigil="remarkup-code-block"><pre class="remarkup-code" style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; padding: 12px; margin: 0; background: rgba(71, 87, 120, 0.08);">[14:48] <sitter> kbroulik: hm, doesn't mousePressEvent get called for all clicks? (i.e. original line 1021-2026 are where the call chain goes into addContainmentActions whcih is where containment_actions is checked), so this implementation would make containment_context_menu=false imply containment_actions=false, no?
[14:49] <kbroulik> kinda
[14:49] <kbroulik> but you can also restrict just containment ations without context menu
[14:49] <kbroulik> oh fuck this shit :(
[14:50] <sitter> but not vice versa (:
[14:50] <sitter> I'd think they are orthogonal from the documentation
[14:50] <sitter> i.e. as previously mentioned one is disable-left/middle-click the other is disable-right-click
[14:51] <kbroulik> so containment_actions should do what my patch doies
[14:51] <kbroulik> and containment_context_menu should burn in hell
[14:51] <sitter> I'd be content with that :)</pre></div>

<p>Or easier to parse perhaps: either <tt style="background: #ebebeb; font-size: 13px;">containment_context_menu</tt> and <tt style="background: #ebebeb; font-size: 13px;">containment_actions</tt> should be orthogonal (applying to right-click and all-other-clicks respectively) OR <tt style="background: #ebebeb; font-size: 13px;">containment_actions</tt> applies to all clicks while <tt style="background: #ebebeb; font-size: 13px;">containment_context_menu</tt> only applies to context menu clicks (right clicks) as suggested by Kai.</p>

<p>Previous documentation on the two is as follows:</p>

<div class="remarkup-code-block" style="margin: 12px 0;" data-code-lang="text" data-sigil="remarkup-code-block"><pre class="remarkup-code" style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; padding: 12px; margin: 0; background: rgba(71, 87, 120, 0.08);">;plasma/containment_actions (since Plasma 4.4)
:Whether or not to allow Plasma mouse actions on containments (usually menus that pop up on mouse clicks).

;plasma/containment_context_menu (4.4)
:Whether a context menu should be shown on containments; this can be used to shut off the desktop context menu in plasma-desktop, for instance.</pre></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R242 Plasma Framework (Library)</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D12380">https://phabricator.kde.org/D12380</a></div></div><br /><div><strong>To: </strong>broulik, Plasma, sitter<br /><strong>Cc: </strong>Frameworks, michaelh, bruns<br /></div>