Skip to content

Pull to Refresh in User Screen#2803

Merged
veloce merged 2 commits intolichess-org:mainfrom
r3econ:refresh-indicator-in-user-screen
Mar 20, 2026
Merged

Pull to Refresh in User Screen#2803
veloce merged 2 commits intolichess-org:mainfrom
r3econ:refresh-indicator-in-user-screen

Conversation

@r3econ
Copy link
Copy Markdown
Contributor

@r3econ r3econ commented Mar 19, 2026

Demo

ios-user-refresh.mov

@ijm8710
Copy link
Copy Markdown

ijm8710 commented Mar 19, 2026

Hey @r3econ thanks for adding this. You're killing it!

Had 2 quick questions for ya if you didn't mind

  1. let's use safari on iOS if you're familiar with it. When you pull to refresh you feel the haptic as you're pulling down. For some reason, maybe it's a result of the app being built in flutter, the haptic in lichess beta, you don't feel until after it's released which feels a little less natural than how it's handled throughout the rest of the iOS ecosystem. And it feels a little weaker/less full. Any chance you notice what I'm saying and is that something the feasibly can be refactored to work more like normal-iOS or because of flutter that's just how the refresh works here?

  2. pull to refresh is great. But Veloce also added a lot of natural refresh dynamically without needing to pull. For example on the home screen once you finish a game, and return back to home, you should see the game listed in 10 recent games without needing to pull. Do you think player profiles should dynamically refresh similarly? I still think it makes sense to have the pull to refresh regardless, but curious your thoughts on the dynamic refresh automatically or you think pull to refresh is good enough. The causality is the data is stale unless a user actions it rather than it always being up to date

Copy link
Copy Markdown
Contributor

@veloce veloce left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I doubt about the usefulness of this feature to be honest. But since you've done it already let's have it.

@veloce veloce merged commit cbe1da9 into lichess-org:main Mar 20, 2026
1 check passed
@r3econ
Copy link
Copy Markdown
Contributor Author

r3econ commented Mar 20, 2026

@ijm8710, good input. I think creating a GitHub Issue for the first point you mentioned would be a good idea. I could definitely look into it and make haptic feedback feel more native

The second is out of scope (I think) since as you say the dynamic refresh is implemented in critical places and for non-critical places a simple pull to refresh will suffice

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

There is a pull refresh for own profile should there be one for other players profiles

3 participants