Skip to content

Performance: Add `LoadingSpinner` delay

Warren Gifford requested to merge tr/loading-spinner-delay into main

Created by: umpox

Actions before merging this PR

  • delay is currently set to always be enabled, we probably don't want this. Change to default false and update usage in our codebase to update it in key places:
    • Key places IMO = In between routes, so we can look at places like here (and more)
  • Experiment with different delay times, we try 300ms here, what does 500ms feel like, and so on. Speak with @quinnkeast about this

Description

Adds a delay to LoadingSpinner so it doesn't render immediately:

App preview:

Check out the client app preview documentation to learn more.

Merge request reports