Skip to content
Snippets Groups Projects

Performance: Add `LoadingSpinner` delay

Open Administrator 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

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading