davidedmundson added a comment.

  > Also, i seem to be missing where you set DBUSMENU_PROPERTY_ID on an action 
(or it's done in d->createAction?).
  
  Yes, it's an existing thing set there.
  
  The map is also existing, and it's used for a few other entry points, 
especially DBusMenuImporter::getMenuById()
  
  I would do your thing in (3), but given this map is already needed, I may as 
well use it.

REPOSITORY
  R120 Plasma Workspace

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

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: davidedmundson, #plasma
Cc: markg, plasma-devel, lesliezhai, ali-mohamed, jensreuterberg, abetts, 
andreaska, sebas

Reply via email to