Transition to full-screen/slideshow mode is too slow

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.

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.

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.

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

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

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

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...

This bug was fixed in the package f-spot - 0.6.1.5-2ubuntu1

---------------
f-spot (0.6.1.5-2ubuntu1) 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