Skip to content

Conversation

@isaaclins
Copy link
Owner

This pull request adds robust support for customizing theme fonts in the application, including both font family and font size. It introduces new fields to the CustomTheme struct, validation logic for these fields, serialization and import/export functionality, and updates the UI to allow users to edit and save theme font settings. Additionally, it ensures that font settings are correctly applied throughout the app and includes comprehensive tests for theme serialization and validation.

Theme font customization:

  • Added font_family and font_size fields to the CustomTheme struct, with defaults and validation logic to ensure valid values and backward compatibility. [1] [2]
  • Updated theme setup logic in ui/theme.rs to apply custom font settings, including parsing font family names and adjusting font sizes for UI elements. [1] [2]

Theme import/export:

  • Implemented methods for exporting and importing custom themes as JSON files, including error handling and user feedback in the delegate. [1] [2]

User interface enhancements:

  • Extended the preferences UI to include controls for editing font family and font size, and added buttons for exporting/importing theme files.

Testing and validation:

  • Added a comprehensive test suite for theme serialization, validation of color and font fields, and backward compatibility with older theme files.

Copilot AI and others added 5 commits November 11, 2025 23:15
Co-authored-by: isaaclins <104733575+isaaclins@users.noreply.github.com>
Co-authored-by: isaaclins <104733575+isaaclins@users.noreply.github.com>
Co-authored-by: isaaclins <104733575+isaaclins@users.noreply.github.com>
Add custom theme export/import with typography support
@isaaclins isaaclins marked this pull request as ready for review November 12, 2025 10:38
isaaclins and others added 7 commits November 12, 2025 11:38
…ilities

Co-authored-by: isaaclins <104733575+isaaclins@users.noreply.github.com>
…ress

Add E2E testing infrastructure for native Rust GUI application
Co-authored-by: isaaclins <104733575+isaaclins@users.noreply.github.com>
…ssue

Add thread accounting for CDN download workers
@isaaclins isaaclins merged commit ce005c3 into main Nov 13, 2025
16 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.

2 participants