Review Request 125973: Remove completely duplicated function

[Martin Klapetek]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/125973/#review88210
-----------------------------------------------------------


By the duplicated function you mean PlasmoidTask::showMenu(int x, int y)? That one has only two args and is missing the "run associated application" action, which I'm not entirely sure what's it used for. The rest indeed looks about the same (though the showMenu could really use some commentary)

- Martin Klapetek


On Nov. 9, 2015, 11:42 p.m., David Edmundson wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/125973/
> -----------------------------------------------------------
> 
> (Updated Nov. 9, 2015, 11:42 p.m.)
> 
> 
> Review request for Plasma.
> 
> 
> Repository: plasma-workspace
> 
> 
> Description
> -------
> 
> Remove almost completely duplicated function.
> One was used for right clicking on expanded items, the other for full compact items.
> 
> 
> Diffs
> -----
> 
>   applets/systemtray/package/contents/ui/ExpandedRepresentation.qml a53be6cb0d9fae14e055a6d1381ad11fee0a236e 
>   applets/systemtray/plugin/host.h db6fc567666281c8b2fe81eab085ae4b950bf07d 
>   applets/systemtray/plugin/host.cpp 61e87050826a28409e44c8dac5baebe12e59bf63 
> 
> Diff: https://git.reviewboard.kde.org/r/125973/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> David Edmundson
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20151110/19180091/attachment.html>