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

Improve loading screens #133

Open
matthiaskern opened this issue Aug 22, 2018 · 8 comments
Open

Improve loading screens #133

matthiaskern opened this issue Aug 22, 2018 · 8 comments
Labels
good first issue Good for newcomers

Comments

@matthiaskern
Copy link
Collaborator

Loading screens should resemble the actual UI it is "morphing" to better.

@matthiaskern matthiaskern added the good first issue Good for newcomers label Aug 22, 2018
@thangngoc89
Copy link
Member

thangngoc89 commented Aug 22, 2018

Are we talking about the loading screens created with ReactContentLoader?

@matthiaskern
Copy link
Collaborator Author

Thinking about these:
image
Could be adjusted to look more like the screen it is loading,e.g.:
image

@thangngoc89
Copy link
Member

yep. that's good ideas. Morphing only works when it looks exactly like the final screen

@nimish-gupta
Copy link
Collaborator

@thangngoc89 can i go forward with this issue?

@thangngoc89
Copy link
Member

@matthiaskern yeah sure. take a look at this thread https://twitter.com/_chenglou/status/1034530454597062656 :P

I found this library with css only approach https://github.com/zalog/placeholder-loading . It looks very promising and we can put the block inside sketch's css to have the exact sizes/spacing

@nimish-gupta
Copy link
Collaborator

@thangngoc89 can you tell me on which of the screen you want loading screen? And provide a little bit more context for the issue?

@thangngoc89
Copy link
Member

@nimish-gupta there is currently 2 loading screens. 1 is the sketch viewer with editor.

https://github.com/Sketch-sh/sketch-sh/blob/9a0b0011d06da4f33b171b806126e13a780ad54b/client/src_editor/Editor_NotePlaceholder.re

And the other one is SketchList (homepage when logined)

module Placeholder = {

I would like to improve them first because currently they are not correct with the final shape (and it's worst then no loading screen at all)

@nimish-gupta
Copy link
Collaborator

ok got it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

3 participants