After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 134342 - window doesn't resize for audio
window doesn't resize for audio
Status: RESOLVED OBSOLETE
Product: totem
Classification: Core
Component: general
unspecified
Other Linux
: Normal minor
: ---
Assigned To: General Totem maintainer(s)
General Totem maintainer(s)
: 157746 161002 310670 311269 318450 322393 331239 437848 532122 (view as bug list)
Depends on:
Blocks: 162815
 
 
Reported: 2004-02-13 18:47 UTC by dobradovic
Modified: 2008-05-08 08:54 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
bvw-hiding-test.patch (1.77 KB, text/plain)
2005-03-09 21:58 UTC, Bastien Nocera
  Details
bvw-hiding-no-audio-or-vfx.patch (6.53 KB, patch)
2005-03-20 10:40 UTC, Bastien Nocera
needs-work Details | Review
gst part (2.34 KB, patch)
2005-04-18 11:38 UTC, Ronald Bultje
committed Details | Review
next try (5.49 KB, patch)
2006-03-09 16:50 UTC, Christoph Burghardt
reviewed Details | Review

Description dobradovic 2004-02-13 18:47:46 UTC
When having the option "adjust window size to new file" activated (rough
translation of mine), and audio visualization deactivated, there is a
problem with audio-files.

1. open video, window adjusts
2. open audio, window remains the same
3. select View->1:!, window resizes correctly (no viewport)

it obviously should resize to no viewport when opening an audio-file I think.
Comment 1 Bastien Nocera 2004-09-20 16:25:32 UTC
Mass reassign.
Comment 2 Keith Lea 2004-10-07 02:07:25 UTC
This might be a duplicate of bug 125903.
Comment 3 Bastien Nocera 2004-11-09 12:32:08 UTC
*** Bug 157746 has been marked as a duplicate of this bug. ***
Comment 4 Bastien Nocera 2004-12-11 12:57:02 UTC
*** Bug 161002 has been marked as a duplicate of this bug. ***
Comment 5 Bastien Nocera 2005-03-09 21:58:02 UTC
Created attachment 38473 [details]
bvw-hiding-test.patch

Test for hiding and showing the BaconVideoWidget. The xine-lib backend seems
quite happy to ignore all that :/
Comment 6 Bastien Nocera 2005-03-20 10:40:57 UTC
Created attachment 38955 [details] [review]
bvw-hiding-no-audio-or-vfx.patch

This will hide the video widget properly.
We still need to handle properly auto-resize on new video sizes, and
setting/unsetting visuals, on the fly.
Comment 7 Ronald Bultje 2005-04-18 11:38:32 UTC
Created attachment 45393 [details] [review]
gst part

This seems to do the trick for the GStreamer backend, with the same remarks as
made above by Bastien.
Comment 8 Christian Kirbach 2005-06-09 21:13:31 UTC
Raising priority to make maintainers aware of patch.
Comment 9 Bastien Nocera 2005-06-09 21:57:39 UTC
The maintainers wrote the patch, and it's not finished...
Comment 10 Sebastien Bacher 2005-07-17 22:27:36 UTC
*** Bug 310670 has been marked as a duplicate of this bug. ***
Comment 11 Bastien Nocera 2005-07-22 16:47:06 UTC
*** Bug 311269 has been marked as a duplicate of this bug. ***
Comment 12 Ronald Bultje 2005-07-22 17:06:53 UTC
(I applied the gst part of the patch a while ago, so marking that as committed.)
Comment 13 Christoph Burghardt 2006-03-09 16:50:16 UTC
Created attachment 60987 [details] [review]
next try

I have made a new patch for this. Auto-resize on new videos and setting / unsetting visuals on the fly is already implemented in CVS HEAD.

This patch tries to disable "hide controls" if necessary so you don't end up with a totem with an empty window. I'm still not happy when playing an audio file in "compact" mode and active the sidebar. Open for suggestions :)
Comment 14 Bastien Nocera 2006-03-13 22:38:42 UTC
*** Bug 318450 has been marked as a duplicate of this bug. ***
Comment 15 Bastien Nocera 2006-03-13 22:40:28 UTC
*** Bug 322393 has been marked as a duplicate of this bug. ***
Comment 16 Bastien Nocera 2006-03-13 22:41:08 UTC
*** Bug 331239 has been marked as a duplicate of this bug. ***
Comment 17 Bastien Nocera 2006-03-13 22:45:16 UTC
From bug 322393:
> Please look at this screenshot :
> http://img292.imageshack.us/my.php?image=screenshottotemmovieplayer0nf.png
> 
> OR, the sidebar could take its place (second screenshot).
> http://img491.imageshack.us/my.php?image=screenshottotemmockup2ej.png

This looks pretty good to me. Christoph, want to have a go at implementing
that? As for your current patch:
- please follow the code style (tabs, not spaces; use spaces before brackets,
etc.)
- pressing 'H' in fullscreen will break it
Comment 18 Stephen 2006-03-29 16:50:24 UTC
Only problem with the mockup is that it won't be a sidebar any more. More a bottombar or topbar :)
Comment 19 Jean-François Fortin Tam 2006-03-29 17:03:52 UTC
Well, it should not be a "sidebar" while we're here (I forgot about that bug):
see bug #322503
Comment 20 Bastien Nocera 2006-08-31 20:25:00 UTC
We only show the logo nowadays, when playing audio only and viz are disabled.
Comment 21 Bastien Nocera 2007-05-16 11:23:31 UTC
*** Bug 437848 has been marked as a duplicate of this bug. ***
Comment 22 Bastien Nocera 2008-05-08 08:54:06 UTC
*** Bug 532122 has been marked as a duplicate of this bug. ***