-
-
Notifications
You must be signed in to change notification settings - Fork 32.3k
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
[Skeleton] Delay the animation by 500ms #18913
Conversation
Details of bundle changes.Comparing: 55e12c9...9c06323
|
Co-Authored-By: Matt <[email protected]>
Co-Authored-By: Matt <[email protected]>
Co-Authored-By: Matt <[email protected]>
Co-Authored-By: Matt <[email protected]>
Co-Authored-By: Matt <[email protected]>
Co-Authored-By: Matt <[email protected]>
This comment has been minimized.
This comment has been minimized.
What's wrong with the style prop? |
Not all knows about the style animation-delay, so just thought it would make it easier to understand the possibility with a prop. |
I can remove the prop, and just make a example in the guide using styles, if thats ok? |
Interesting, I was wondering about the motivation, thanks. So your main concern is about discoverability. |
Yes
|
We could make the same argument for a few other component props that only apply a single CSS property. Is there a good rule of thumb for when to have a prop vs using style? |
@mbrookes Not I'm aware of. I think that the higher the probability of needing the behavior, the more we should consider having a built-in prop. |
This seems like a pretty strong use case then. |
is it possible to restart a build, when it fails on connection to host github.com? |
Sounds good I'll dig in to it. :)
tor. 19. dec. 2019 15.44 skrev Olivier Tassinari <[email protected]>:
… ***@***.**** commented on this pull request.
------------------------------
In docs/src/pages/components/skeleton/skeleton.md
<#18913 (comment)>:
> @@ -33,3 +33,7 @@ The component supports 3 variants.
## Facebook example
{{"demo": "pages/components/skeleton/Facebook.js", "defaultCodeOpen": false, "bg": true}}
+
+## Delay example
This is the only study I could find:
https://uxdesign.cc/what-you-should-know-about-skeleton-screens-a820c45a571a.
It suggests that we could support a wave and pulsating animation. This
could be an interesting improvement.
Regarding the delay, I think that we should give up on the demo and new
props, to reduce the number of animation on the screen. The new demo
personally gives me motion sickness, I don't know about you guys.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#18913>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABZEJL6GG4IU45MILITR5HTQZOCDJANCNFSM4J4KFUGA>
.
|
Thank you for the contribution, I wish we could backed this change with a specific user study, the closest we have to justify the changes is https://www.nngroup.com/articles/response-times-3-important-limits/. |
Thx, i kinda think we have, since i read it (skimming), an choose 0.5s after reading this.
|
Improve the case where the skeleton is shown for a short period of time, it reduces the mental overhead of seeing an animation, if the skeleton is shown for less than 500ms, it will feels like the content is static.