fix: smooth drawer animation and respect animation preferences

[jasi381]

Summary

• Fixes Animation in the sidebar when changing accounts is too fast and hectic #10480 — Sidebar animation too fast/hectic
• Increases drawer animation duration from 200ms to 300ms with FastOutSlowInEasing for a smoother feel
• Adds isShowAnimations to DrawerConfig so the drawer respects the user's "Show animations" setting
• When animations are disabled (via app settings or OS-wide), all drawer transitions use snap() (instant, no animation) instead of slide/fade

Test plan

• Open sidebar and toggle the account switcher — animation should feel smooth (not jarring)
• Go to Settings → disable "Show animations" → open sidebar and toggle account switcher — transitions should be instant (no animation)
• Set Developer Options → Window/Transition animation scale = 0 → verify drawer transitions are instant
• Verify folder list expand/collapse icon still works correctly
• Verify switching between accounts animates the account name smoothly

[dani-zilla]

The detekt issue should be a simple fix:

Task :feature:navigation:drawer:dropdown:detekt FAILED
/home/runner/work/thunderbird-android/thunderbird-android/feature/navigation/drawer/dropdown/src/main/kotlin/net/thunderbird/feature/navigation/drawer/dropdown/ui/common/AnimatedExpandIcon.kt:15:14: Parameters in a composable function should be ordered following this pattern: params without defaults, modifiers, params with defaults and optionally, a trailing function that might not have a default param.
Current params are: [isExpanded: Boolean, isShowAnimations: Boolean = true, modifier: Modifier = Modifier, tint: Color? = null] but could be [isExpanded: Boolean, modifier: Modifier = Modifier, isShowAnimations: Boolean = true, tint: Color? = null].

Basically just swapping the isShowAnimations: Boolean = true to be after the modifier param.

Besides that, would you be willing to make a screen recording of this issue? I seem to be having trouble reproducing the initial issue to test the fix.

[Alecaddd]

Thank you so much for this contribution.
I would recommend to simply listen for the OS-wide animation disabled state rather than adding an extra option inside the application.
From a UX point of view, it feels weird to only have an app with disabled animations but not the OS, or vice versa, so I think this feature should be always in sync with the OS and respect the global users' choice.

[jasi381]

Thanks for the feedback!

For the animation improvements, I adjusted the drawer animation to make it smoother by increasing the duration from 200ms to 300ms and applying FastOutSlowInEasing so the motion feels less abrupt.

I'm currently addressing the Detekt parameter order issue mentioned by @dani-zilla and will push the fix shortly.

Regarding the UX suggestion from @Alecaddd, I’m also working on updating the implementation so the app respects the OS-wide animation preference instead of relying on a separate app-level option.

I've attached short screen recordings below showing the behaviour before and after the animation changes to help with reproducing the issue.

Before fix (original behaviour):

before.mp4

After animation improvement:

after.mp4

[untitaker]

Just glanced over the code and didn't test it, but this fix seems to make the drawer respect system settings, but the application setting ("Use gaudy") still exists with an independent default. This means that all non-drawer animations still do not respect the system-wide settings. I think the correct fix is to refactor the setting (not any views) to a triple-state where the default is to inherit the system-wide setting. I'm working on that fix right now.

[github-actions]

Missing report label. Set exactly one of: report: include, report: exclude OR report: highlight.