View Single Post
gidzzz's Avatar
Posts: 282 | Thanked: 2,387 times | Joined on Sep 2011
#1270
Originally Posted by gidzzz View Post
If the playback manager makes a good use of provided foreign PIDs, than the cleanest approach would be to rewrite the widget, but I still have to test it and I don't know if I'd have enough time for another project.

Doing what the stock player does wouldn't play well with "Do nothing" set as the exit action or Nicolai's widget, but fortunately the stock widget appears to send different things on D-Bus when there's playback going on, so I'll try to make a use of that.
I did more tests and I think it's impossible to get this to work nicely without rewriting the widget. Using widget's PID when interacting with libplayback does not help anything, but that's not the worst problem. The widget declares that it finished playing only if you press its pause button, not even when the playlist ends. What it means is that whatever happens, unless you press widget's pause button, sooner or later you will encounter a situation in which you have to press the play button one extra time. It is perhaps possible to monitor D-Bus for widget's activity and use the collected information to decide when to take over, but I think it's too much effort to only please the desktop widget.

Instead, I have added an option to act in a way similar to the stock player ("Compatible" method) and it's enabled by default, because it can be disputable which behavior is better. "Reliable" method is what was in the previous version and it is called that way because it doesn't silence notifications based on presses of the play button, but on the actual playing state, so it doesn't get confused even if you change the state using another application while OMP is open. Although "Reliable" will cause the music to be stopped after clicking the stock widget, it should be less of a pain with today's update, because Now Playing will be opened, like in the stock player.

That said, it might be a good moment to list all changes for today's update.
  • Dropped the hack behind "Headset button action" at the cost of making this feature CSSU-only, but then it works also with "Prevent playback interruptions".
  • Added an option to choose a method of silencing notifications.
  • Added Dark Lyrics plugin.
  • Fixed UTF-8 in AZLyrics plugin.
  • Fixed problems with opening some streams.
  • Improved feedback when opening streams by bouncing the buffering bar.
  • Fixed outdated buffering info sometimes being shown after changing a station.
  • Prevented detached and dead windows from appearing when activating the playback indicator and something else shortly thereafter.
  • open_mp_now_playing_playback_on and open_mp_radio_playing_playback_on (used by the desktop widget) open the appropriate window.
  • Added the missing shortcut to close the album art menu.
  • Optified the package with debugging symbols.
  • Updated translations.

From now on, if you want to use "Headset button action", you need policy-settings-rx51 modified by Pali, which also remedies issues with "Prevent playback interruptions" enabled at the same time. policy-settings-rx51 0.4.10.8+0m5 is a part of CSSU T-maemo5, but if you don't want to install/update CSSU, installing the package from http://repository.maemo.org/communit...settings-rx51/ manually might also work.



Edit: Clicked the submit button too soon.

Originally Posted by sixwheeledbeast View Post
One thing I noticed today was that settings for the headset are only updated when the headset is removed and then reinserted.
With "Prevent playback interruptions" skipping worked that way and pause/resume was completely broken. With latest OMP and CSSU there shouldn't be any problems.


Originally Posted by toxaris View Post
Found a strange thing too, dont know if its just me.

Well, when I add new albums the thumbnails isnt imported in the All Album view.
But if I play the album the thumbnail is there.
If the files are not named "cover.jpg", Tracker won't pick them up. If you play a song without a cover, OMP looks by itself for "cover.jpg" (in case Tracker failed for some reason or the cover was removed from the cache), "front.jpg" and "folder.jpg". Hopefully it explains what's going on.
__________________
My Thumb mini-repository: http://gidzzz.mooo.com/maemo/.

Last edited by gidzzz; 2012-09-12 at 17:05.
 

The Following 16 Users Say Thank You to gidzzz For This Useful Post: