Frontend tweaks#189
Merged
luk27official merged 4 commits intocusbg:mainfrom Dec 27, 2025
Merged
Conversation
There was a problem hiding this comment.
Pull request overview
This PR implements frontend improvements including package updates, a loading spinner during structure overpainting, and log file viewing for failed tasks. The changes span backend API endpoints for log retrieval, executor modifications to copy logs on failure, and frontend UI enhancements.
- Updated frontend dependencies to latest versions
- Added loading spinner UI during polymer color changes
- Implemented log file access for failed docking and tunnels tasks via new API endpoints
Reviewed changes
Copilot reviewed 13 out of 14 changed files in this pull request and generated 16 comments.
Show a summary per file
| File | Description |
|---|---|
| web-server/src/tunnels_task.py | Added get_log method to retrieve log files for tasks by hash |
| web-server/src/docking_task.py | Added get_log method, fixed regex raw string, changed type hint from any to typing.Any, added null check for TaskInfo.data |
| web-server/src/api_v2.py | Added API routes for retrieving docking and tunnels task logs |
| gateway/Dockerfile | Updated Node.js base image from version 19.7 to 24 |
| frontend/server/server.develop.js | Changed route handler from app.get to app.use for better middleware compatibility |
| frontend/package.json | Updated all frontend dependencies to latest versions |
| frontend/client/viewer/components/visualization-tool-box.tsx | Made polymer color change async with loading state to show spinner during overpainting |
| frontend/client/viewer/components/visualization-tool-box.css | Added CSS styles for loading overlay spinner |
| frontend/client/viewer/components/tasks-table.tsx | Added clickable failed task status with log viewing functionality |
| frontend/client/viewer/application.tsx | Made onPolymerColorChange async to support loading state, added type annotation for RCSB callback |
| frontend/build/webpack.common.js | Added fullySpecified: false to resolve ES module import issues |
| executor-tunnels/run_task.py | Added logic to copy log files to public directory when tasks fail |
| executor-docking/run_task.py | Added logic to copy log files to public directory when tasks fail |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Features: