--519c26eba083462e866ce8c274339a07
Content-Transfer-Encoding: 7bit
Content-Type: text/plain; charset="ascii"
Mime-Version: 1.0

xuetianweng created this revision.
xuetianweng added reviewers: Plasma, davidedmundson.
Restricted Application added a project: Plasma.

REVISION SUMMARY
  Right now, new Action always appends to the list, but application may
  want to add new action in the middle of the menu. This breaks such
  application with dynamic menu entries. This change tries to keep the
  order of actions same as the order in dbus request.

TEST PLAN
  Manually tested the brokened menu in certain applications.

REPOSITORY
  R120 Plasma Workspace

BRANCH
  Plasma/5.11

REVISION DETAIL
  https://phabricator.kde.org/D8585

AFFECTED FILES
  libdbusmenuqt/dbusmenuimporter.cpp

To: xuetianweng, #plasma, davidedmundson
Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart

--519c26eba083462e866ce8c274339a07
Content-Transfer-Encoding: quoted-printable
Content-Type: text/html; charset="ascii"
Mime-Version: 1.0

<table><tr><td style=3D"">xuetianweng created this revision.<br />xuetianwe=
ng added reviewers: Plasma, davidedmundson.<br />Restricted Application add=
ed a project: Plasma.
</td><a style=3D"text-decoration: none; padding: 4px 8px; margin: 0 8px 8px=
; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; back=
ground-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1=
f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href=
=3D"https://phabricator.kde.org/D8585" rel=3D"noreferrer">View Revision</a>=
</tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>Right now,=
 new Action always appends to the list, but application may<br />
want to add new action in the middle of the menu. This breaks such<br />
application with dynamic menu entries. This change tries to keep the<br />
order of actions same as the order in dbus request.</p></div></div><br /><d=
iv><strong>TEST PLAN</strong><div><p>Manually tested the brokened menu in c=
ertain applications.</p></div></div><br /><div><strong>REPOSITORY</strong><=
div><div>R120 Plasma Workspace</div></div></div><br /><div><strong>BRANCH</=
strong><div><div>Plasma/5.11</div></div></div><br /><div><strong>REVISION D=
ETAIL</strong><div><a href=3D"https://phabricator.kde.org/D8585" rel=3D"nor=
eferrer">https://phabricator.kde.org/D8585</a></div></div><br /><div><stron=
g>AFFECTED FILES</strong><div><div>libdbusmenuqt/dbusmenuimporter.cpp</div>=
</div></div><br /><div><strong>To: </strong>xuetianweng, Plasma, davidedmun=
dson<br /><strong>Cc: </strong>plasma-devel, ZrenBot, progwolff, lesliezhai=
, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>
--519c26eba083462e866ce8c274339a07--