Skip to content

Restore table spinner - #1679

Open
Arnei wants to merge 2 commits into
opencast:r/20.xfrom
Arnei:fix-table-spinner
Open

Arnei wants to merge 2 commits into
opencast:r/20.xfrom
Arnei:fix-table-spinner

Conversation

@Arnei

@Arnei Arnei commented Sep 24, 2026 •

Copy link
Copy Markdown
Member

Show a spinning spinner icon while loading data into a table (for the first time). Apparently we had that at some point, but it silently broke. This restores it.

Also restores spinning functionality to all spinners (they were not spinning).

How to test this

Switch between tables. More visible with larger tables.

AI Usage

Claude Sonnet 5 was used with this PR.

The spin animation was lost when fontawesome was removed, so no loading or running icon rotated anymore.

This commit was AI generated
Read the loading status of the table's resource slice and keep showing existing rows while newer data loads.

This commit was AI generated
@Arnei Arnei added the type:bug Something isn't working label Sep 24, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Use docker or podman to test this pull request locally.

Run test server using develop.opencast.org as backend:

podman run --rm -it -p 127.0.0.1:3000:3000 

Specify a different backend like stable.opencast.org:

podman run --rm -it -p 127.0.0.1:3000:3000 -e PROXY_TARGET=https://stable.opencast.org 

It may take a few seconds for the interface to spin up.
It will then be available at http://127.0.0.1:3000.
For more options you can pass on to the proxy, take a look at the README.md.

@github-actions

Copy link
Copy Markdown
Contributor

This pull request is deployed at test.admin-interface.opencast.org/1679/2026-09-24_14-08-03/ .
It might take a few minutes for it to become available.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type:bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant