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 page load/unload experience (remove FOUCs) #599

Merged
merged 3 commits into from
Feb 25, 2020

Conversation

NBTX
Copy link
Contributor

@NBTX NBTX commented Feb 25, 2020

Various improvements to the UI in terms of page loading/unloading.

@ghost
Copy link

ghost commented Feb 25, 2020

DeepCode's analysis on #6962d2 found:

  • 0 critical issues. ⚠️ 1 warning and 1 minor issue. ✔️ 0 issues were fixed.

💬 This comment has been generated by the DeepCode bot, installed by the owner of the repository. The DeepCode bot protects your repository by detecting and commenting on security vulnerabilities or other critical issues.


☺️ If you want to provide feedback on our bot, here is how to contact us.

font-display: swap;
// Do not use font-display: swap for the icon font - it looks really bad when the page
// loads.
font-display: block;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oops! I realized this was the reason for #462
Thank you @NBTX, I never knew this!

@TravisBuddy
Copy link

Hey @NBTX,
Your changes look good to me!

View build log

TravisBuddy Request Identifier: 6ad64750-5766-11ea-821d-47621684b1e8

@TravisBuddy
Copy link

Hey @NBTX,
Your changes look good to me!

View build log

TravisBuddy Request Identifier: 8f1d8a60-5766-11ea-821d-47621684b1e8

liyasthomas
liyasthomas previously approved these changes Feb 25, 2020
Copy link
Member

@liyasthomas liyasthomas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@TravisBuddy
Copy link

Hey @NBTX,
Your changes look good to me!

View build log

TravisBuddy Request Identifier: 0efc65f0-576f-11ea-b14f-b186f6b55800

@dihmeetree
Copy link
Contributor

Awesome @NBTX :)

@liyasthomas liyasthomas merged commit a8c380c into master Feb 25, 2020
@liyasthomas liyasthomas deleted the fix/fouc-page-load-unload branch February 25, 2020 03:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants