<table><tr><td style="">subdiff created this revision.<br />subdiff added projects: Plasma, Frameworks.<br />Restricted Application added a subscriber: Frameworks.
</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/D4921" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>With <a href="https://phabricator.kde.org/D4867" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: line-through;" rel="noreferrer">D4867</a> the placement was already changed a bit, but mostly the correction if there is not enough screen space available. This new patch is meant to harmonize the placement for the available enums in a sensible way.</p>

<p>Instead of much talking take a look at the placement <strong>prior</strong> to <a href="https://phabricator.kde.org/D4867" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: line-through;" rel="noreferrer">D4867</a>:<br />
<a href="https://phabricator.kde.org/F2676516" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;" rel="noreferrer">F2676516: plan1.png</a></p>

<p>Currently / with <a href="https://phabricator.kde.org/D4867" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: line-through;" rel="noreferrer">D4867</a>:<br />
<a href="https://phabricator.kde.org/F2676517" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;" rel="noreferrer">F2676517: plan2.png</a></p>

<p>With this patch:<br />
<a href="https://phabricator.kde.org/F2676518" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;" rel="noreferrer">F2676518: plan3.png</a></p>

<p>Regarding the enum names in the API and the normal use cases of a menu I think this last pattern makes the most sense. In this case the correctional behavior on screen space limits could get simplified as well. This would become a separate patch though.</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>Tested with plasma-pa and task manager context menu.</p></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/D4921" rel="noreferrer">https://phabricator.kde.org/D4921</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>src/declarativeimports/plasmacomponents/qmenu.cpp</div></div></div><br /><div><strong>To: </strong>subdiff, Plasma<br /><strong>Cc: </strong>Frameworks, plasma-devel, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol<br /></div>