Skip to content
This repository has been archived by the owner on Jul 6, 2020. It is now read-only.

about "pull to refresh" #782

Closed
kklem0 opened this issue May 2, 2015 · 2 comments
Closed

about "pull to refresh" #782

kklem0 opened this issue May 2, 2015 · 2 comments

Comments

@kklem0
Copy link

kklem0 commented May 2, 2015

#781 (comment)

It's a valid on going discussion and when I'm gathering info you just locked it, is that really how grown up you are??
What I was trying to write was the implementation clearly looks wrong from what you described, pull to refresh should not be able to get triggered by scrolling, but by "dragging", as it should not be a part of the webpage, but be triggered by actions, for example like this one http://apeatling.com/demos/web-ptr/ .

@BernhardPosselt
Copy link
Contributor

Read the rest of the issue. It's not pull to refresh. Please open a new issue with a proper suggestion

@jancborchardt
Copy link
Contributor

@clementhk see my comment at #781 (comment)

One problem though with the exampe you linked is that dragging is not discoverable at all. It needs to work via scrolling, but maybe only after you scrolled to the top and scroll again. It’s a tough nut to crack on desktop and I can understand why @Raydiation is annoyed about it.

@clementhk let’s maybe try to find a good implementation and experiment with it in ownCloud Mail. Then we might be wiser.

@kklem0 kklem0 changed the title Ohh please, you did not just locked a discussion that you don't like, Kim Jong-un? about "pull to refresh" May 2, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants