Transition to full-screen/slideshow mode is too slow

Bug #513045 reported by David Siegel on 2010-01-27
This bug affects 4 people
Affects Status Importance Assigned to Milestone
Fix Released
One Hundred Papercuts
f-spot (Ubuntu)

Bug Description

It takes nearly three seconds to enter full screen mode or slide show mode after pressing the button in F-Spot's toolbar. This transition is excessive, and it makes the cross-fade effect seem long and drawn-out.

(Furthermore, pressing escape during this long transition causes a NullReferenceException which crashed F-Spot. This is a separate bug, but shows that F-Spot is not rock solid during this transition.)

Related branches

Changed in hundredpapercuts:
milestone: none → lucid-round-7
Matt Perry (mp) wrote :

This patch changes the full screen fade from 3 seconds to 1 second. 0.6 seconds would have been ideal but the method requires an integer.

Vish (vish) on 2010-01-27
Changed in hundredpapercuts:
importance: Undecided → Low
status: New → Triaged
Changed in f-spot (Ubuntu):
importance: Undecided → Low
status: New → Triaged
Nat Wilson (njwilson23) wrote :

Frankly, I think it would be nice to have an option to turn off the transition effects altogether. It looks cool the two times you use it, but to me becomes annoying after that.

I suppose that this would be a feature request beyond the papercuts project. Thanks for working on speeding it up though.

Martin Lindhe (martinlindhe) wrote :

I agree with Nat. But perhaps can this papercut be solved simply with an ability to "disable pretty effects" checkbox setting?

We should just fix the slow transition and not create a new checkbox.
What is needed here is a better default behavior. F-Spot should strike
the right balance between "pretty effects" and performance -- surely
every application shouldn't add an "ugly mode" toggle ;)

The 3-second transition is egregious, so turning the transition off
would also be an acceptable solution if we cannot simply speed it up.

Sebastien Bacher (seb128) wrote :

Thank you for your work there, I'm unsubscribing the sponsors since the change seems to still need to some work to get the correct value, I will review the hundredpapercut round and upload changes before beta if we don't get anything better by then though

tags: added: patch
tags: added: patch-needswork
removed: patch
Alex Launi (alexlauni) wrote :

Here's a patch to allow double values, so that a 0.6 value can be used for entering full screen mode.

Vish (vish) on 2010-02-21
tags: added: patch
removed: patch-needswork
Sebastien Bacher (seb128) wrote :

Thank you for your work, could you add your change to the upstream bug too?

Changed in f-spot (Ubuntu):
status: Triaged → Fix Committed
Changed in hundredpapercuts:
status: Triaged → Fix Committed
Sebastien Bacher (seb128) wrote :

Did you try the change? The second part of the change lacks a ";", the first one doesn't work either to the DoubleAnimation() change apparently, not sure why but I will let somebody who does C# to figure this one or have a look later when I've time to debug this one. I will change it to use 1 second for now...

Launchpad Janitor (janitor) wrote :

This bug was fixed in the package f-spot -

f-spot ( lucid; urgency=low

  * debian/patches/git_copy_attribute_normal.patch:
    - git change to set the copy attributes to normal, fix a write permission
      issue (lp: #483268)
  * debian/patches/git_no_urlencode.patch:
    - git change to not urlencode the mail subjects (lp: #454972)
  * debian/patches/git_correct_description.patch:
    - git change to fix the profiles description displayed (lp: #502744)
  * debian/patches/git_color_profile_printing_breakage.patch:
    - git change to fix printing when a color profile is selected (lp: #496134)
  * debian/patches/git_key_conflict.patch:
    - git change to fix a shortcut conflict (lp: #516801)
  * debian/patches/git_set_default_resize_value.patch:
    - git change to set a correct default resize value (lp: #513864)
  * debian/patches/ubuntu_edit_tag.patch:
    - change the edit tag label to be consistent with other entries
      (lp: #513452)
  * debian/patches/ubuntu_transition_timing.patch:
    - change by Alex Launi to make the transition effect between modes take 1s
      (lp: #513045)
  * debian/patches/ubuntu_slider_animation_tweaking.patch:
    - change by Alex Launi to reduce the slider animation timing (lp: #513004)
  * debian/patches/ubuntu_check_duplicate.patch:
    - change by Matt Perry to use a better label for the duplicates checking
      (lp: #511767)
 -- Sebastien Bacher <email address hidden> Mon, 01 Mar 2010 16:06:41 +0100

Changed in f-spot (Ubuntu):
status: Fix Committed → Fix Released
Vish (vish) on 2010-03-03
Changed in hundredpapercuts:
status: Fix Committed → Fix Released
Changed in f-spot:
status: Unknown → Fix Released
Changed in f-spot:
importance: Unknown → Medium
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.