Review Request 120005: Plugins specific settings

[Ronny Yabar Aizcorbe]

> On Aug. 30, 2014, 7:04 p.m., Aleix Pol Gonzalez wrote:
> >

As I pointed out, my goal was to create per-plugin settings. If you change the value hasSettings to true, it creates another row in the Settings called: $Plugin Settings that opens PluginSettingsActivity. 

The only requirement is to create a file plugin_$name_preferences like xml/plugin_mpris_preferences.xml

So what's your suggestion?


> On Aug. 30, 2014, 7:04 p.m., Aleix Pol Gonzalez wrote:
> > src/main/res/values/strings.xml, line 76
> > <https://git.reviewboard.kde.org/r/120005/diff/2/?file=308640#file308640line76>
> >
> >     Is that really something you'd want as a configuration? Maybe the UI should just be extended?
> 
> Albert Vaca Cintora wrote:
>     I like the patch to be able to have settings per-plugin, and even though I agree with you that this doesn't make much sense as a setting, I can't see a way to put it in the normal UI. Do you have any idea?
> 
> Aleix Pol Gonzalez wrote:
>     A long tap could open a dialog asking how much it needs to jump, for example.

Yes, because maybe in the future, there will be other settings to add.


- Ronny


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


On Aug. 30, 2014, 11:04 a.m., Ronny  Yabar Aizcorbe wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/120005/
> -----------------------------------------------------------
> 
> (Updated Aug. 30, 2014, 11:04 a.m.)
> 
> 
> Review request for kdeconnect.
> 
> 
> Repository: kdeconnect-android
> 
> 
> Description
> -------
> 
> Added support for specific plugin UI settings. Added interval time preferences to fast forward or rewind a multimedia file instead of harcording the time.
> 
> 
> Diffs
> -----
> 
>   src/main/AndroidManifest.xml 499e92e 
>   src/main/java/org/kde/kdeconnect/Plugins/BatteryPlugin/BatteryPlugin.java 7761c21 
>   src/main/java/org/kde/kdeconnect/Plugins/ClibpoardPlugin/ClipboardPlugin.java 68315ff 
>   src/main/java/org/kde/kdeconnect/Plugins/MousePadPlugin/MousePadPlugin.java 491e694 
>   src/main/java/org/kde/kdeconnect/Plugins/MprisPlugin/MprisActivity.java b219e07 
>   src/main/java/org/kde/kdeconnect/Plugins/MprisPlugin/MprisPlugin.java 3884392 
>   src/main/java/org/kde/kdeconnect/Plugins/NotificationsPlugin/NotificationsPlugin.java adc4825 
>   src/main/java/org/kde/kdeconnect/Plugins/PingPlugin/PingPlugin.java 51d2028 
>   src/main/java/org/kde/kdeconnect/Plugins/Plugin.java a91016a 
>   src/main/java/org/kde/kdeconnect/Plugins/PluginFactory.java 8d046cc 
>   src/main/java/org/kde/kdeconnect/Plugins/SftpPlugin/SftpPlugin.java eee7bf9 
>   src/main/java/org/kde/kdeconnect/Plugins/SharePlugin/SharePlugin.java b0c28cb 
>   src/main/java/org/kde/kdeconnect/Plugins/TelephonyPlugin/TelephonyPlugin.java d26409a 
>   src/main/java/org/kde/kdeconnect/UserInterface/PluginSettingsActivity.java PRE-CREATION 
>   src/main/java/org/kde/kdeconnect/UserInterface/SettingsActivity.java e1599c5 
>   src/main/res/values/arrays.xml PRE-CREATION 
>   src/main/res/values/strings.xml bec4a22 
>   src/main/res/xml/plugin_mpris_preferences.xml PRE-CREATION 
> 
> Diff: https://git.reviewboard.kde.org/r/120005/diff/
> 
> 
> Testing
> -------
> 
> Checked and tested. Works well.
> 
> 
> Thanks,
> 
> Ronny  Yabar Aizcorbe
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdeconnect/attachments/20140904/6a802512/attachment.html>