Review Request: Icontasks: Add (hidden) config-option to manually set media-applications

Commit Hook null at kde.org
Sat Sep 22 06:49:17 UTC 2012


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/105137/#review19280
-----------------------------------------------------------


This review has been submitted with commit 846c819fb18a10350d92fef2cdaa5521a12b284f by Anton Kreuzkamp to branch master.

- Commit Hook


On June 2, 2012, 7:57 p.m., Anton Kreuzkamp wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/105137/
> -----------------------------------------------------------
> 
> (Updated June 2, 2012, 7:57 p.m.)
> 
> 
> Review request for Plasma and Craig Drummond.
> 
> 
> Description
> -------
> 
> This patch adds a option to the config-file to manually set media apps, for applications that provide an mpris interface, but don't belong into the categories AudioVideo or Music, like rekonq (with this patch: https://git.reviewboard.kde.org/r/105059/). Rekonq is also added to the default-config.
> 
> This patch actually contains 3 commits: 1. Fix a bug that made the mediabuttons.cpp never be read (due to a wrong filepath) 2. Add the config-option 3. Add rekonq to the customMediaApps in the default config.
> 
> I guess it cannot be seen as a bugfix (didn't correctly find all media-applications) and so has to wait for 4.10, right?
> 
> 
> Diffs
> -----
> 
>   applets/icontasks/mediabuttons.h 00fbfd8 
>   applets/icontasks/mediabuttons.cpp dff97fb 
>   applets/icontasks/mediabuttonsrc 37e7d91 
> 
> Diff: http://git.reviewboard.kde.org/r/105137/diff/
> 
> 
> Testing
> -------
> 
> Tested with rekonq, works like a charm. No problems I could think of.
> 
> 
> Thanks,
> 
> Anton Kreuzkamp
> 
>

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


More information about the Plasma-devel mailing list