search

yaxim-org / yaxim

yaxim - a lean XMPP/Jabber client for Android
https://yaxim.org
GNU General Public License v3.0
486 stars 238 forks source link

Feature Request: Option for static notification in tray #123

Closed jfrederickson closed 10 years ago

jfrederickson commented 10 years ago

I recently got a smartwatch, and one of its features is to forward all notifications from my phone to the watch. That's great, except that Yaxim updates the notification whenever it disconnects and reconnects, which is... all the time where I am. Turning off the tray icon isn't really a good option either, as then Yaxim falls prey to the OOM killer far more often than I'd like.

I propose an option to have a static notification in the tray, maybe something that just says "running" - enough to keep Yaxim alive most of the time, but not enough to bombard my wrist with notifications. It's a bit of a hack-ish workaround, I'll admit, but barring native support for the smartwatch's API (which I really can't expect you to do) it's the only way I can think of to fix this.

ge0rg commented 10 years ago

I have been asked already about exactly this kind of support for smart watches. Is there a common API or intent annotation I can use without much overhead?

In slightly related news, I have re-worked the notification to provide more useful information (APK). Feel free to test and continue to blame me about the sheer amount of notifications. I will not add an option for what you wish, but I might change the notification to completely remove the "ticker text" which scrolls by. I hope this would be filtered away by your smart smartwatch.

jfrederickson commented 10 years ago

Sadly there is not a common API to my knowledge, but I'll look into it. Smartwatches unfortunately are very different - the only common feature between most of them is notification forwarding.

I can definitely understand that you wouldn't want to add such an ugly hack to your application, and thanks for the response anyway. The new build's notifications are much more useful. In addition, after a bit of further testing, it appears that my watch in particular only displays notifications with ticker text. I hadn't thought about it since most notifications do have ticker text, but then there are some (Google Now, other passive notifications) that don't. So consider me in favor of that change.

jfrederickson commented 10 years ago

And of course, just as I say there's no common API, Google announces their own official one: Android Wear. Of course it looks like it's another closed platform like Google Glass, and it won't work with any existing smartwatches, but... possibly worth looking into? (Some info about voice replies here.)

That probably deserves a separate request though as it's getting pretty far out of the scope of this one. And since you've made it clear that you won't be implementing this one, and since it was somewhat misguided in the first place as I hadn't done enough testing to know what I really wanted... I'll go ahead and close this. Less clutter in the bug tracker. :)

ge0rg commented 10 years ago

Issue is fixed in yaxim 0.8.7b. Download APK.