💡 Add option to disable pull-to-refresh

[mouadlotfi]

Description

Currently, Fladder has pull-to-refresh enabled on most screens (dashboard, library, details pages, etc.). While this is a useful feature for many users, there's no way to disable it for those who find it triggers accidentally during normal scrolling.

Requested Feature

Add a toggle setting in Settings → Visual to enable/disable pull-to-refresh functionality across the app.

Use Case

• Users who accidentally trigger refresh while scrolling
• Desktop users who don't need/want pull-to-refresh behavior
• Users who prefer manual refresh (F5 key already works for refresh)

Suggested Implementation

Add a new boolean setting in ClientSettingsModel:

@Default(true) bool enablePullToRefresh

Then modify the PullToRefresh widget to conditionally wrap content with RefreshIndicator based on this setting:

child: settings.enablePullToRefresh && widget.onRefresh != null
    ? RefreshIndicator(...)
    : widget.child(context)

Benefits

• Gives users control over UI behavior
• Prevents accidental refreshes
• Maintains current behavior by default (enabled)
• F5 keyboard shortcut would still work for manual refresh

Platform

All platforms (Windows, Linux, macOS, Android, iOS)

Additional Context

The pull-to-refresh is currently implemented in lib/widgets/shared/pull_to_refresh.dart and used across many screens. A single setting would control this behavior app-wide.

[schembriaiden]

I can't see this being useful to anyone.

On desktop I am yet to see a person use pull to scroll on anything.

On a laptop which can fold and become a tablet it is also not useful because you cannot access the F5 key like you would on any normal keyboard.

On mobile/tablet for me the pull to refresh does not seem near sensitive enough to be done by mistake (I would say the pull to refresh currently needs to be forced to be done), and either way how would you refresh manually then if you can't pull?

Only place I can see this being somewhat useful is if you have a laptop with a touchscreen and you like to use the touchscreen to scroll, which I would say is also a very small percentage of people.

[mouadlotfi]

I can't see this being useful to anyone.

On desktop I am yet to see a person use pull to scroll on anything.

On a laptop which can fold and become a tablet it is also not useful because you cannot access the F5 key like you would on any normal keyboard.

On mobile/tablet for me the pull to refresh does not seem near sensitive enough to be done by mistake (I would say the pull to refresh currently needs to be forced to be done), and either way how would you refresh manually then if you can't pull?

Only place I can see this being somewhat useful is if you have a laptop with a touchscreen and you like to use the touchscreen to scroll, which I would say is also a very small percentage of people.

I understand it may not be useful to you, but some users would appreciate the option to disable it. The setting would default to enabled, so nothing changes for anyone who wants to keep it.

[PartyDonut]

I would have to agree with @schembriaiden a bit here. But also because I'm not sure what this would solve. It's not just a pull-to-refresh it also acts as the load indicator.

I'm also curious does it activate that quickly for you?

Under normal circumstances you can scroll up and down without it activating. I have to explicitly scroll to the top and then drag again for it to activate.

[mouadlotfi]

I would have to agree with @schembriaiden a bit here. But also because I'm not sure what this would solve. It's not just a pull-to-refresh it also acts as the load indicator.

I'm also curious does it activate that quickly for you?

Under normal circumstances you can scroll up and down without it activating. I have to explicitly scroll to the top and then drag again for it to activate.

I appreciate the feedback! I personally just don't like the behavior and would love the option to turn it off. I'm not asking for it to be removed, just a simple toggle so users like me can disable it :)