Time slider does no more advance after closing preferences window

mario0alberto1 / gnome-mplayer

Automatically exported from code.google.com/p/gnome-mplayer

0 stars 0 forks source link

Time slider does no more advance after closing preferences window #634

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago

What steps will reproduce the problem?
1. Play an audio file
2. Open preferences window
3. Close preferences window

What is the expected output?
The time slider should advance.

What do you see instead?
The time slider does no more advance.

What version of the product are you using? On what operating system?
Version 1.0.6 on Arch Linux with PulseAudio.

Original issue reported on code.google.com by thomas.s...@gmail.com on 30 Aug 2012 at 5:13

GoogleCodeExporter commented 8 years ago

Haen't seen that before. What version of mplayer, and does switching the ao to 
alsa change anything?

Original comment by kdeko...@gmail.com on 30 Aug 2012 at 11:30

GoogleCodeExporter commented 8 years ago

$ pacman -Qi mplayer-vaapi | grep version
Version        : 35014-1

Switching Audio Output to ALSA does not change anything.
Are you able to reproduce this?

Original comment by thomas.s...@gmail.com on 31 Aug 2012 at 1:50

GoogleCodeExporter commented 8 years ago

I actually can reproduce it. Funny that it does not happen with video files, as 
far as I know everything should be the same. Actually none of the buttons work 
after restart so something is going wrong there.

Original comment by kdeko...@gmail.com on 31 Aug 2012 at 2:03

GoogleCodeExporter commented 8 years ago

Found it and fixed it in gmtk, SVN version 183

This patch should apply to v1.0.6 of gmtk as well, as long as you remove the 
gm_log line.

http://code.google.com/p/gmtk/source/detail?r=183

Very simple, it just moves the state change out of the if statement.

Original comment by kdeko...@gmail.com on 31 Aug 2012 at 2:15

GoogleCodeExporter commented 8 years ago

Thanks for the fast fix.
I can confirm it is working now.

Original comment by thomas.s...@gmail.com on 31 Aug 2012 at 2:37

GoogleCodeExporter commented 8 years ago

Original comment by kdeko...@gmail.com on 4 Sep 2012 at 10:03

Changed state: Fixed

GoogleCodeExporter commented 8 years ago

Issue 522 has been merged into this issue.

Original comment by kdeko...@gmail.com on 4 Sep 2012 at 10:06

GoogleCodeExporter commented 8 years ago

Issue 648 has been merged into this issue.

Original comment by kdeko...@gmail.com on 19 Oct 2012 at 4:18

GoogleCodeExporter commented 8 years ago

Issue 648 has been merged into this issue.

Original comment by kdeko...@gmail.com on 20 Oct 2012 at 1:01