-
Notifications
You must be signed in to change notification settings - Fork 32
🎨 [Frontend] UX Enh: Starting osparc #7987
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR enhances the dashboard UX by showing per-tab loading messages, deferring hiding until all resources load, and introducing a context‐aware “No resources found” label.
- Show a custom “Loading …” page in each browser before fetching resources and hide it in promise
.finally(). - Add a
noResourcesFoundlabel toResourceContainerManagerwith context‐specific messages. - Invoke
evaluateNoResourcesFoundLabelin Tutorial, Study, and App browsers to display when no items exist.
Reviewed Changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| TutorialBrowser.js | Show “Loading Tutorials…” and hide loading page in .finally(). |
| StudyBrowser.js | Show “Loading Projects…” and coordinate hide via Promise.all(). |
| AppBrowser.js | Show “Loading Apps…” and hide loading page after services/hypertools load. |
| ResourceContainerManager.js | Add noResourcesFound label and evaluateNoResourcesFoundLabel() method. |
| Dashboard.js | Minor formatting: add missing semicolon after initResources() call. |
services/static-webserver/client/source/class/osparc/dashboard/ResourceContainerManager.js
Show resolved
Hide resolved
services/static-webserver/client/source/class/osparc/dashboard/ResourceContainerManager.js
Show resolved
Hide resolved
|
@mergify queue |
✅ The pull request has been merged automaticallyThe pull request has been merged automatically at b4006b9 |
sanderegg
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
|



What do these changes do?
This PR improves the UX of the "Starting osparc" message.
Related issue/s
How to test
Dev-ops