Skip to content

Bug fixes - #2

Merged
romer8 merged 11 commits into
mainfrom
bugFixes
Feb 2, 2026
Merged

Bug fixes#2
romer8 merged 11 commits into
mainfrom
bugFixes

Conversation

@romer8

@romer8 romer8 commented Feb 2, 2026

Copy link
Copy Markdown
Collaborator

This pull request introduces several improvements and fixes to the charting and data management components in the React application, focusing on better theme handling, improved UI responsiveness, and enhanced state management for asynchronous actions. The most significant changes are grouped by theme below.

Chart Theming and Styling Improvements:

  • Added explicit CSS variables for individual chart line colors (--chart-line-0 through --chart-line-4) in both light and dark themes in App.scss, allowing for more flexible and maintainable color theming. [1] [2]
  • Updated the chart rendering logic in Plot.js to use the new CSS variables for line colors, and refactored style variable retrieval for better performance and maintainability.
  • Simplified the static SVG layer in the chart by removing unused axis and grid rendering code, and improved axis styling and tick rendering. [1] [2]

UI Responsiveness and User Feedback:

  • Enhanced the cache table component to provide responsive feedback during deletion operations: added local loading state, disabled buttons during async actions, and displayed visual indicators for ongoing deletions. [1] [2] [3]
  • Updated the chart and cache table containers to use styled components for more consistent UI and easier future customization. [1] [2]

State Management and Async Handling:

  • Improved the variables menu to handle concurrent requests safely using refs and request IDs, preventing race conditions and ensuring only the latest async result is applied. Also, integrated with the global VPU store for variable data. [1] [2] [3]
  • Updated the data menu to use the correct selected feature ID from the global feature store, fixing a bug where the wrong feature could be referenced. [1] [2] [3]

Developer Experience:

  • Added helpful logging and comments for cache key and S3 prefix generation in the data menu to aid debugging and traceability.

These changes collectively improve the maintainability, reliability, and user experience of the application's data visualization and management features.

@romer8
romer8 merged commit 25b5682 into main Feb 2, 2026
0 of 4 checks passed
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.

1 participant