Skip to content
This repository has been archived by the owner on Jun 24, 2022. It is now read-only.

Improve loading #2595

Merged
merged 1 commit into from
Mar 28, 2022
Merged

Improve loading #2595

merged 1 commit into from
Mar 28, 2022

Conversation

anxolin
Copy link
Contributor

@anxolin anxolin commented Mar 27, 2022

Summary

Add loading a blinking effect while loading for the "Loading ..." message.

After adding Lazy-loading and code-splitting we were showing this message. This PR is just a small improvement on handing the app boot.

Mar-27-2022.15-06-21.mp4
Mar-27-2022.15-08-40.mp4

To Test

Just throttle the internet a little bit

image

Observe loading message is now loading

@anxolin anxolin requested review from a team March 27, 2022 14:10
@elena-zh
Copy link

Hey @anxolin , could you please help to generate the PR-preview link?

@W3stside
Copy link
Contributor

fix the history please

@anxolin anxolin changed the base branch from develop to release/1.12.0 March 28, 2022 10:14
@anxolin
Copy link
Contributor Author

anxolin commented Mar 28, 2022

@elena-zh I will merge to consolidate.

We will generate a link including this and the latest other things so u can double check it there. Here it didn't work the PRaul

@anxolin anxolin merged commit 0a874c5 into release/1.12.0 Mar 28, 2022
@elena-zh
Copy link

Hey @anxolin , I have verified this PR in the release PR.
Loading effect looks nicer! However, I will leave #2554 task opened in order to make this loading indicator even better in the future.
Currently, in WIN 10 with a slow internet connection this 'Loading' text looks a bit weird due to the #970 issue
fonts.jpg
So I think we should enhance it somehow in the future.

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

Successfully merging this pull request may close these issues.

3 participants