F-Spot screensaver too fast

Bug #387656 reported by Allan on 2009-06-16
24
This bug affects 4 people
Affects Status Importance Assigned to Milestone
F-Spot
Fix Released
Medium
One Hundred Papercuts
Undecided
Unassigned
f-spot (Ubuntu)
Undecided
Iain Lane

Bug Description

The F-Spot screensaver cycles through pictures much too quickly (~1/second). Can we dial down the speed?

Related branches

Changed in hundredpapercuts:
status: New → Confirmed
Changed in hundredpapercuts:
milestone: none → round-8
Ted M Lin (kitlaan) wrote :

In src/SlideView.cs, there seem to be two variables that control the slideshow transitions:
* flip_interval (hardcoded to 2000 milliseconds)
* transition_interval (hardcoded to 75 milliseconds)

Some user testing will be needed to determine what values to use, but the default values do seem to be a bit quick.

kbogert (kb-turnkeyprogramming) wrote :

Quick, simple patch to enable changing flip_interval and transition_interval using gconf (/apps/f-spot/ui/slideshow_flip_interval and /apps/f-spot/ui/slideshow_transition_interval both need to be string-type keys, sorry)

A few comments on your patch;
 - I don't think upstream like that you add and remove white lines
 - Why did you have to have string values? Can't you cast cast to integer instead of string as Get () return an object?
 - I think we'll need to supply some gconf schemas as well

(note I'm no f-spot nor C# developer so I might be wrong here)

Andrew (and471) wrote :

This is the times for the 'Pictures Slideshow' I think we should go by these

#define N_FADE_TICKS 10
#define MINIMUM_FPS 3.0
#define IMAGE_LOAD_TIMEOUT 10000

there's no upstream bug for this. if you want this fixed and released for karmic, please file a bug.

re #2:
- there's a Preferences class you have to use to get any preference. it proxies the preference request to the backend in use (most probably GConf)

re #3:
- upstream does not really care about white lines. it happens.
- you're right here
- no, we don't use schema

komputes (komputes) wrote :

Every screensaver shipped has hidden settings capabilities, not attainable through gnome-screensaver.

I vote on importing functionality from the xscreensaver package into the gnome-screensaver package to modify screensaver parameters.

Two small issues with gaving gnome-screensaver and xscreensaver installed:
1) Currently you need to kill gnome-screensaver to run xscreensaver or else it will not run. Short of uninstalling gnome-screensaver I am not sure how to simply disable it (I would have thought it could be done "System > Preferences > Startup Applications" but it's not there.

2) gnome-screensaver and xscreensaver have the same .desktop file and tooltip, so when thay are both installed you cannot tell them apart in the applications menu.

Andrew (and471) wrote :

@komputes

if you look here : http://live.gnome.org/GnomeScreensaver_2fFrequentlyAskedQuestions

You will see the gnome-screensaver does not want to offer the ability to change settings of screensavers. Whether we agree with that or not (I don't) it means that upstream is not going to coordinate on this.

Therefore the solution of this bug is just to increase the hard-coded values

Andrew (and471) wrote :

This has been fixed in upstream GIT (there is now a dialog which you can change these parameter with, it's an extension) and will be in f-spot 6.1 Hopefully we can get this packaged.

Andrew (and471) on 2009-08-24
Changed in f-spot:
importance: Undecided → Unknown
status: New → Unknown
status: Unknown → Fix Committed
Changed in f-spot:
status: Fix Committed → Fix Released

rugby471, if we can't package the new F-Spot, will you please look into patching the hardcoded value that is currently used? 5 seconds would be better than 1 second, and I'm sure there would be no problem merging the change before UI freeze.

Andrew (and471) on 2009-08-26
Changed in f-spot (Ubuntu):
status: New → In Progress
Changed in hundredpapercuts:
status: Confirmed → In Progress
Andrew (and471) wrote :

f-spot 0.6.1 is going into karmic, Ian Lane is packaging it

Changed in f-spot (Ubuntu):
assignee: nobody → Iain Lane (laney)
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package f-spot - 0.6.1.1-0ubuntu1

---------------
f-spot (0.6.1.1-0ubuntu1) karmic; urgency=low

  * Sync on debian with updated requirement to build on karmic
  * debian/control, debian/rules:
    - use a gnome-screensaver build-depends rather than a rules workaround

  [ Iain Lane ]
  * New Upstream Version 0.6.1.1 (LP: #418936)
    - some files were missing from the tarball
    - fix SendEmail for thunderbird3beta
  * New Upstream Version 0.6.1.0
    - fix some crashers
    - reduce the both the package size and installed size by a few Megs
    - works on gtk+ > 2.17
    - load .pcx files
    - extendable slideshow
    - ~30 bugs fixed
      + screensaver too fast (LP: #387656)
      + crash with ArgumentNullException when deleting a photo (LP: #290654)
      + Should remember "copy photos to Photos folder" setting (LP: #270238)
      + Crash when opening settings dialog (Closes: #531301)
    - updated translations
  * debian/patches/debian_link-system-flickrnet.patch,
    debian/patches/debian_link-system-gnome-keyring.patch
    debian/patches/debian_link-system-mono-addins.patch
    debian/patches/ubuntu_nofuse_fix_photo_import.patch: Update to apply cleanly
  * debian/patches/ubuntu_xdg-photo-dir.patch: Add new patch from Andrew
    Higginson, slightly modified, to default to update the user's default
    directory to XDG_PICTURES_DIR/Photos (localised) as the location to import
    pictures. Only applies to new users; existing users will see no change.
    (LP: #127315)
  * debian/rules: Remove leftover CDBS variables
  * debian/rules: Move f-spot-screensaver to /usr/lib/xscreensaver so it
    actually works. Previously it was installed in /usr/libexec and so was not
    found by gnome-screensaver. (Closes: #544204)
  * debian/control, debian_link-system-flickrnet.patch: Update flickrnet
    dependency to new ABI (Closes: #543803)
  * debian/control: Increase Standards-Version to 3.8.3, no changes required
  * debian/copyright: Update bundled library locations, point to versioned GPL
    and update FSF address

 -- Sebastien Bacher <email address hidden> Tue, 01 Sep 2009 21:37:53 +0200

Changed in f-spot (Ubuntu):
status: In Progress → Fix Released
Andrew (and471) on 2009-09-02
Changed in hundredpapercuts:
status: In Progress → Fix Released
Grizzly (sven-witterstein) wrote :

This bug is still in lucid -> F-Spot Screensaver switches every second or so.

Changed in f-spot:
importance: Unknown → Medium
r000t (vistro) wrote :

This is horrible.... I have no time to enjoy the image. #12 above me is correct.

To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.