Skip to content

Add skeleton loader to database subNavigation while tables load#2673

Merged
stnguyen90 merged 3 commits into
mainfrom
copilot/add-skeleton-loader-to-subnav
Dec 11, 2025
Merged

Add skeleton loader to database subNavigation while tables load#2673
stnguyen90 merged 3 commits into
mainfrom
copilot/add-skeleton-loader-to-subnav

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Dec 6, 2025

The subNavigation sidebar flashes "No tables yet" while tables are being fetched, creating a jarring UX when the API call takes time.

Changes

  • Add loading state initialized to true, set to false after API call completes
  • Import Skeleton component from @appwrite.io/pink-svelte
  • Show skeleton loaders (3 rows mimicking table icon + name) during initial load
  • Only show "No tables yet" after loading completes with zero results
{#if loading}
    <Layout.Stack gap="s">
        {#each Array(3) as _}
            <Layout.Stack gap="s" direction="row" alignItems="center">
                <Skeleton variant="line" width={16} height={16} />
                <Skeleton variant="line" width="70%" height={16} />
            </Layout.Stack>
        {/each}
    </Layout.Stack>
{:else if tables?.total}
    <!-- table list -->
{:else}
    No tables yet
{/if}

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • o1.ingest.sentry.io
    • Triggering command: /opt/hostedtoolcache/node/20.19.6/x64/bin/node node /home/REDACTED/work/console/console/node_modules/.bin/../vitest/vitest.mjs run (dns block)
    • Triggering command: /opt/hostedtoolcache/node/20.19.6/x64/bin/node node build.js (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

Original prompt

On subNavigation.svelte, In src/routes/(console)/project-[region]-[project]/databases/database-[database]/subNavigation.svelte, while the tables are loading, the sub navigation shows "No tables yet". In cases where it takes a little bit of time to fetch the collections, the flash of "No tables yet" and then the data showing can be jarring.

We should show a skeleton loader while it's loading for the first time.


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

@appwrite
Copy link
Copy Markdown

appwrite Bot commented Dec 6, 2025

Console (appwrite/console)

Project ID: 688b7bf400350cbd60e9

Sites (1)
Site Status Logs Preview QR
 console-stage
688b7cf6003b1842c9dc
Ready Ready View Logs Preview URL QR Code

Tip

Functions can run for up to 15 minutes before timing out

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Dec 6, 2025

Important

Review skipped

Bot user detected.

To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.


Comment @coderabbitai help to get the list of available commands and usage tips.

Co-authored-by: stnguyen90 <1477010+stnguyen90@users.noreply.github.com>
Copilot AI changed the title [WIP] Add skeleton loader for tables in subNavigation Add skeleton loader to database subNavigation while tables load Dec 6, 2025
Copilot AI requested a review from stnguyen90 December 6, 2025 00:16
@stnguyen90
Copy link
Copy Markdown
Contributor

stnguyen90 commented Dec 8, 2025

Result:

Screen.Recording.2025-12-08.at.14.07.18.mov

@stnguyen90 stnguyen90 requested a review from ItzNotABug December 8, 2025 21:48
@stnguyen90 stnguyen90 marked this pull request as ready for review December 8, 2025 22:08
@stnguyen90 stnguyen90 merged commit 557a0cd into main Dec 11, 2025
4 checks passed
@stnguyen90 stnguyen90 deleted the copilot/add-skeleton-loader-to-subnav branch December 11, 2025 18:51
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.

3 participants