Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Wait 2 sec before showing spining loaders #2116

Closed
elatif2020 opened this issue Sep 25, 2023 · 5 comments · Fixed by #2119
Closed

Wait 2 sec before showing spining loaders #2116

elatif2020 opened this issue Sep 25, 2023 · 5 comments · Fixed by #2119
Assignees

Comments

@elatif2020
Copy link
Collaborator

Currently we have spinning loaders in several places:

In the navbar
On the whole (feed+chat) screen before the layout appears
In the feed\inbox bar (also after pinned items are loaded)
In the chat bar before anything is loaded there

To make the experience feel better we should wait 2 sec (after the action \ after screen appears) before showing our spinning loader

@andreymikhadyuk
Copy link
Collaborator

@elatif2020 you wrote after the action, what do you mean under that?

@andreymikhadyuk
Copy link
Collaborator

@elatif2020 I do not see spinning loader in the navbar, we display skeleton there. The spinning loader is in the sidenav.

@andreymikhadyuk
Copy link
Collaborator

andreymikhadyuk commented Sep 26, 2023

@elatif2020 so, we should add 2 sec delay in spinning loader display in the following places:

  • in the sidenav
  • on the center of the screen screen (feed+chat) before the layout appears
  • in loading the feed items (even on the loading of the nexts batches?)
  • in the chat

@andreymikhadyuk
Copy link
Collaborator

@elatif2020 I've added delay to the following loaders:
Screenshot 2023-09-26 at 12 49 02
image

also:

  • to the loader of next batches
  • to the loader of chat messages loading

@elatif2020
Copy link
Collaborator Author

elatif2020 commented Oct 12, 2023

We might have to do this again for the Feed page on mobile #2193

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

Successfully merging a pull request may close this issue.

2 participants