sketch-sh icon indicating copy to clipboard operation
sketch-sh copied to clipboard

Improve loading screens

Open matthiaskern opened this issue 6 years ago • 8 comments

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

matthiaskern avatar Aug 22 '18 20:08 matthiaskern

Are we talking about the loading screens created with ReactContentLoader?

thangngoc89 avatar Aug 22 '18 20:08 thangngoc89

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

matthiaskern avatar Aug 23 '18 06:08 matthiaskern

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

thangngoc89 avatar Aug 23 '18 06:08 thangngoc89

@thangngoc89 can i go forward with this issue?

nimish-gupta avatar Aug 31 '18 13:08 nimish-gupta

@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

thangngoc89 avatar Aug 31 '18 14:08 thangngoc89

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

nimish-gupta avatar Sep 01 '18 14:09 nimish-gupta

@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) https://github.com/Sketch-sh/sketch-sh/blob/4c7fafec4784a4b8a9dda290f9ad3a0df2561857/client/src/components/UI_SketchList.re#L94

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)

thangngoc89 avatar Sep 01 '18 14:09 thangngoc89

ok got it

nimish-gupta avatar Sep 01 '18 14:09 nimish-gupta