Merged
Conversation
- Update CSS variables to use Nebula dark blue accent (#1F4E8C) - Replace purple/violet colors with Nebula color palette: - Background: #121317 (very dark, near-black) - Surface: #1E2130 (dark charcoal) - Text primary: #E0E6F0 - Text secondary: #A3A9BF - Divider: #333A56 - Set dark theme as default preference in theme manager - Update gradients and glass morphism effects - Replace violet/purple classes in HTML with blue equivalents - Maintain accessibility and contrast ratios Addresses GitHub issue #43 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
The workout section was hidden by default with a 'hidden' class but was never being shown when workout data was loaded. This prevented the entire workout visualization panel from being displayed. Changes: - Remove 'hidden' class from workoutSection when displaying workout - Ensure workoutInfo panel is properly shown with both class removal and style setting - Add null checks for better error handling Fixes the issue where loading sample workout or uploading files would not show the main power profile chart. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
|
Looks like there are a few issues preventing this PR from being merged!
If you'd like me to help, just leave a comment, like
Feel free to include any additional details that might help me get this PR into a better state. You can manage your notification settings |
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.
Summary
Complete implementation of Nebula UI's dark blue accent and dark theme foundations, replacing the current purple/violet color scheme to align with the internal power-user design system.
Changes Made
CSS Variables Update (styles.css)
--primaryfrom#4f46e5to#1F4E8C(Nebula dark blue)--backgroundto#121317(very dark, near-black)--surfaceto#1E2130(dark charcoal)--text-primaryto#E0E6F0--text-secondaryto#A3A9BF--dividerto#333A56Theme Manager Updates (themes.js)
applyDarkTheme()with complete Nebula color paletteapplyLightTheme()to maintain Nebula consistencyTailwind Config Updates (index-modern.html)
Testing
Acceptance Criteria Met
Visual Impact
This change transforms the entire UI from a purple/violet theme to a sophisticated dark blue Nebula theme that:
Breaking Changes
None - all functionality remains intact while updating the visual theme.
Fixes #43
🤖 Generated with Claude Code