<div dir="ltr">The background shouldn't be white on *every* phone, it should use the system's default background color.</div><div class="gmail_extra"><br><div class="gmail_quote">On Sat, Feb 14, 2015 at 1:25 AM, Chandramouleshwar S <span dir="ltr"><<a href="mailto:mouli.4543@gmail.com" target="_blank">mouli.4543@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div>Actually I set the background to white. When i tested in motoX, the icon background was black including the text...so nothing was visible. And about the icon, i set the icon which appears in the top notification panel to app icon...earlier it was used to be 'play'<br></div> button icon. I made other changed too. <br></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><div class="gmail_quote">On Sat, Feb 14, 2015 at 7:43 AM, Albert Vaca <span dir="ltr"><<a href="mailto:albertvaka@gmail.com" target="_blank">albertvaka@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">I haven't been able to test it yet, sorry. The only thing I noticed when I checked the code review, is that it says that you changed "the background and icon of the notification" but it doesn't seem to be anything in the code for that. Did you forget to commit the interface xml files?<br></div><div><div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Feb 12, 2015 at 4:43 AM, Chandramouleshwar S <span dir="ltr"><<a href="mailto:mouli.4543@gmail.com" target="_blank">mouli.4543@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi...I made some changes in feature/noti-mediacontrol branch regarding aforementioned issues and submitted a patch. It would be great if someone reviews it suggest improvements as its not reviewed yet. I see a small bug in 'onConnectionLost' function and working on that now.<br></div><div><div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Feb 5, 2015 at 3:37 AM, Albert Vaca <span dir="ltr"><<a href="mailto:albertvaka@gmail.com" target="_blank">albertvaka@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div><div>IMO we should implement both the old and the new APIs, because we don't want to support only Android 5 devices. Nobody is working on this branch so feel free to do the changes that you want. The issues that we found and that made me decide to roll back the changes from the master branch are:<br></div>- The media player notification was never dismissed from the panel, even if you stop the music player. So once it appeared it would keep showing there until you kill kdeconnect.<br></div>- Other apps didn't see kdeconnect as a media player, so they couldn't interact with it. Specifically, I have a pebble watch and I thought I should be able to control kdeconnect through it, but that didn't work.<br>- The media player notification lost the reference to the media player control plugin when the plugin was reloaded. This was specially notable because there was a bug were plugins would be reloaded every time you launch the kdeconnect app, even though this bug is now fixed in the master branch (you might want to merge this branch back into the noti-mediacontrol branch for this).<span><font color="#888888"><br><br></font></span></div><span><font color="#888888"><div>Albert<br></div><div><br></div></font></span></div><div><div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Feb 4, 2015 at 1:57 PM, Chandramouleshwar S <span dir="ltr"><<a href="mailto:mouli.4543@gmail.com" target="_blank">mouli.4543@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">I checkedout that branch and the code want compiling. So i changed a line and compiled it succesfully. Can i know what are the issues with it and why is it not added to the master branch. I see that you are using RemoteControlClient to display notification on the lock screen. I saw that its deprecated in API 21. I would like to use the newer supported MediaSession and accomplish this. Is anyone working on it. I am new here, so it would be great if someone helps me in getting around.<br></div><div><div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Feb 2, 2015 at 9:47 PM, vineet garg <span dir="ltr"><<a href="mailto:grgvineet@gmail.com" target="_blank">grgvineet@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi <span style="font-size:13px;white-space:nowrap">Chandramouleshwar</span><span style="font-size:13px;font-weight:bold;white-space:nowrap">,</span><div><span style="white-space:nowrap">That feature is already implemented by </span><span style="color:rgb(51,51,51);font-family:arial,'lucida grandriale','lucida sans unicode',tahoma,sans-serif;line-height:16.5px;text-align:center">Da-Jin Chu but removed from </span><span style="color:rgb(51,51,51);font-family:arial,'lucida grandriale','lucida sans unicode',tahoma,sans-serif;line-height:16.5px;text-align:center">master branch due to some issues. You should check feature/noti-mediacontrol branch.</span></div><div><span style="color:rgb(51,51,51);font-family:arial,'lucida grandriale','lucida sans unicode',tahoma,sans-serif;line-height:16.5px;text-align:center"><br></span></div><div><span style="color:rgb(51,51,51);font-family:arial,'lucida grandriale','lucida sans unicode',tahoma,sans-serif;line-height:16.5px;text-align:center">As there is always a room for improvement, so may be you could try that .</span></div><div><span style="color:rgb(51,51,51);font-family:arial,'lucida grandriale','lucida sans unicode',tahoma,sans-serif;line-height:16.5px;text-align:center"><br></span></div><div><span style="color:rgb(51,51,51);font-family:arial,'lucida grandriale','lucida sans unicode',tahoma,sans-serif;line-height:16.5px;text-align:center">Vineet Garg</span></div><div><div><br></div></div></div><div class="gmail_extra"><br><div class="gmail_quote"><div><div>On Mon, Feb 2, 2015 at 9:28 PM, Chandramouleshwar S <span dir="ltr"><<a href="mailto:mouli.4543@gmail.com" target="_blank">mouli.4543@gmail.com</a>></span> wrote:<br></div></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div><div dir="ltr">I would like to add a notification tray that shows player controls when music is playing in kde( like how google play music shows play controls on notification try). Is somebody working on this right now?. If not, I wish to go on with the implementation. <br clear="all"><span><font color="#888888"><div><div><br>-- <br><div><div dir="ltr">With Peace,<div>Chandramouleshwar S</div><div><br></div></div></div>
</div></div></font></span></div>
<br></div></div>_______________________________________________<br>
KDEConnect mailing list<br>
<a href="mailto:KDEConnect@kde.org" target="_blank">KDEConnect@kde.org</a><br>
<a href="https://mail.kde.org/mailman/listinfo/kdeconnect" target="_blank">https://mail.kde.org/mailman/listinfo/kdeconnect</a><br>
<br></blockquote></div><br></div>
</blockquote></div><br><br clear="all"><br>-- <br><div><div dir="ltr">With Peace,<div>Chandramouleshwar S</div><div><br></div></div></div>
</div>
</div></div><br>_______________________________________________<br>
KDEConnect mailing list<br>
<a href="mailto:KDEConnect@kde.org" target="_blank">KDEConnect@kde.org</a><br>
<a href="https://mail.kde.org/mailman/listinfo/kdeconnect" target="_blank">https://mail.kde.org/mailman/listinfo/kdeconnect</a><br>
<br></blockquote></div><br></div>
</div></div></blockquote></div><br><br clear="all"><br>-- <br><div><div dir="ltr">With Peace,<div>Chandramouleshwar S</div><div><br></div></div></div>
</div>
</div></div></blockquote></div><br></div>
</div></div></blockquote></div><br><br clear="all"><br>-- <br><div><div dir="ltr">With Peace,<div>Chandramouleshwar S</div><div><br></div></div></div>
</div>
</div></div></blockquote></div><br></div>