plotly / plotly/dash

Implement a better initial loading state

Open
#633 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

feature P3
Dominant language
Python
Stars
24.4k
Forks
2.3k
Avg merge
2d 7h
Merged PRs (30d)
13

Description

Right now, the text "Loading..." appears before React gets instantiated, as it is in the HTML index string here: https://github.com/plotly/dash/blob/05545461214e43707e7d4e36485935dc166be987/dash/dash.py#L61 - because we now have a different way of handling loading with the new Loading component and loading states, we should also implement a better way of handling initial loading. We could add a default spinner instead of the text "Loading...", for example. We can still keep the _dash-loading div around for users that use their own CSS spinners.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start in dash/dash.py at the HTML index string around line 61, and inspect how the _dash-loading div and the newer Loading component are handled. Replace the initial text presentation with a default spinner while retaining the div for users’ custom CSS spinners; done means the initial page shows the improved loading state before React starts.

Written by the indexing model from the issue text.

Assessment

Tech stack
python, react
Domain
frontend
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.