Search code examples
javascriptreactjsasynchronousindexeddbdexie

React - show hide two elements without flickering on page load


Edit. I rewrote the code to be even more minimalist. The below code is a spike test of my issue.

Here is a video of the issue:

https://i.sstatic.net/EhItK.jpg

I have two components.

The first component is named TextEditor (and it is a text editor) but its content is irrelevant - the component could be anything. A simple div with text would be just as relevant.

The next component is named Workflows and is used to render a collection from IndexDB (using the Dexie.js library). I named this collection "workflows" and the state variable I store them in is named workflows_list_array

What I am trying to do is the following:

When the page loads, I want to check if any workflows have a specific ID . If they do, I store them in workflows_list_array and render them. I don't need help with this part.

However, if no workflows with the aforementioned criteria exist, I want to keep the component named Workflows hidden and render the component named TextEditor. If workflows do exist, I want the TextEditor hidden and to display Workflows

The problem is that even though I have it working, when workflows do exist (when workflows_list_array is populated) the TextEditor "flickers" briefly before being hidden and then the Workflows component is displayed.

I can tell this is an async issue but I can't tell how to fix it.

I posted code below and I tried to keep it to a minimum.

Test.js

import React, {useState, useEffect} from "react";
import db from "../services"

function Workflows(props){
    return (

          <div>
             <ul>
               {
                props.workflows.map((val,index)=>{

                     return <li key={index}>{val.content}</li>
                })
               }
             </ul>
          </div>
    )
}


function TextEditor(){

    return (

          <div> TextEditor </div>
    )
}


function Test(props){
   let [workflows_list_array, set_state_of_workflows_list_array] = useState([]);

   let [client_id_number, set_client_id_number] = useState(5);
      useEffect(() => { // get all workflows of the selected client per its ID

         db.workflows.toArray((workflows_list)=>{    // iterate through workflows array
               return workflows_list

         }).then((workflows_list)=>{

             workflows_list.forEach((val)=>{

                   if(client_id_number === val.client_id){
                       set_state_of_workflows_list_array((prev)=>{
                          return [...prev, val]
                       });
                   }
             });

         });


    }, []);



    return(
        <div>
          {workflows_list_array.length ? null : <TextEditor/> }
          {workflows_list_array.length ? <Workflows workflows={workflows_list_array}/> : null}

        </div>
    )
}


export default Test

services.js

import Dexie from 'dexie';
import 'dexie-observable';


var workflowMagicUserDB = new Dexie("WorkflowMagicUserDB");

workflowMagicUserDB.version(1).stores({
    user: "",
    workflows: "++id,client_id,content,title",
    clients: "++id,name",
    calendar_events: "++id,start,end,title"
});

export default workflowMagicUserDB

Solution

  • Why don't you include a flag which indicates if you have already got data from IndexDB, something like:

    function Test(props){
       const [loading, setLoading] = React.useState(true);
       let [workflows_list_array, set_state_of_workflows_list_array] = useState([]);
    
       let [client_id_number, set_client_id_number] = useState(5);
          useEffect(() => {
             db.workflows.toArray((workflows_list)=>{
             }).then((workflows_list)=>{
             }).finally(() => setLoading(false)); //when process finishes, it will update the state, at that moment it will render TextEditor or Workflows
        }, []);
    
       if(loading) return <LoadingIndicator/>; // or something else which indicates the app is fetching or processing data
    
        return(
            <div>
              {workflows_list_array.length ? null : <TextEditor/> }
              {workflows_list_array.length ? <Workflows workflows={workflows_list_array}/> : null}
    
            </div>
        )
    }
    
    
    export default Test
    

    When the process finishes, finally will be executed and set loading state to false, after that, your app will render TextEditor or Workflows