🎨 Fix Exchange Configuration GUI Dark Theme Styling - #18
Merged
Conversation
- Created setup.py with dynamic version generation from git commit - Fixed version format from invalid '2026.02.21-e713ca2' to compliant '2026.2.21.dev0+commit' - Added robust error handling for git operations - Successfully builds both source distribution (.tar.gz) and wheel (.whl) - Resolves CI/CD pipeline packaging failures - Supports console script entry point for 'powertrader' command
- Remove hardcoded 'robinhood' default from pt_hub.py primary_exchange - Add handling for empty primary_exchange to prevent credential errors - Set US region default exchanges to disabled instead of enabling robinhood - Prevents CI failures when GitHub secrets/credentials aren't configured - Allows graceful operation without requiring immediate exchange setup - Resolves integration test failures in CI/CD pipeline
- Move credential loading from import-time to runtime in pt_trader.py - Replace SystemExit(1) with graceful error handling - Add POWERTRADER_ENV=test check to skip credentials in CI - Credentials now loaded only when CryptoAPITrading class is instantiated - Prevents build failures when GitHub secrets/credentials aren't configured - Allows basic import testing without requiring exchange credentials This follows proper CI/CD practices where builds shouldn't need external credentials
- Make KuCoin client imports optional in pt_thinker.py and pt_trainer.py - Skip KuCoin initialization in POWERTRADER_ENV=test environments - Add proper error handling for when market client is unavailable - Move pt_thinker initialization from import-time to main block - Prevents CI/CD failures from pkg_resources dependency issues in kucoin-python Key changes: - Conditional KuCoin imports with graceful fallback - Import-time network calls moved to runtime only - Compatible with Python 3.13+ where pkg_resources was deprecated - Maintains full functionality in production environments
…tion - Add DataProvider abstraction layer to decouple core modules from specific exchanges - Create comprehensive exchange configuration with 66+ platforms across: * Centralized exchanges (28): Tier 1/2 + regional (Rain, Yellow Card, Quidax, VALR) * DeFi/DEX platforms (18): Ethereum, Layer2, lending, staking protocols * Derivatives (7): perpetuals, options, synthetics * Specialized platforms (8): prediction markets, P2P, NFT/gaming * Cross-chain infrastructure (5): bridges and aggregators - Update pt_thinker.py and pt_trainer.py to use provider interface - Enable regional exchange optimization (US/EU/Asia/Africa-MENA priorities) - Implement intelligent fallback strategies across entire ecosystem - Restore user choice in exchange selection vs KuCoin lock-in Resolves architectural issue where sophisticated multi-exchange system was bypassed by hardcoded dependencies in core prediction modules.
- Changed version format from '2026.02.22-b7f563a' to PEP 440 compliant '2026.02.22+b7f563a' - Added separate PYTHON_VERSION output for packaging compatibility - Fixes InvalidVersion error: 'Invalid version: 2026.02.22-b7f563a' in setup.py - Uses '+' for local version identifier instead of '-' as per PEP 440
- Implement comprehensive dark theme matching main PowerTrader Hub - Fix Setup Exchange tab color inversion issues - Replace ttk widgets with tk widgets for proper dark backgrounds - Apply consistent dark color scheme across all 66 exchanges - Improve text contrast and widget visibility - Enhance professional appearance with proper borders and highlights Resolves exchange configuration GUI theming inconsistencies
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
1 participant
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.
🎨 Exchange Configuration GUI Dark Theme Fixes
Problem
The Exchange Configuration GUI had significant color inversion and theming issues, particularly on the "Setup Exchange" tab where widgets appeared with incorrect backgrounds and poor contrast.
Solution
Technical Changes
ttk.Framewithtk.Framefor proper background controlVisual Improvements
#070B10)#C7D1DB) for excellent readability#00FF66) for highlights#243044)Testing
Screenshots
The Exchange Configuration GUI now seamlessly integrates with the main application's professional dark theme, providing a consistent and visually appealing user experience across all 66 supported cryptocurrency exchanges.