Skip to content

Add theme support#77

Draft
Blake-goofy wants to merge 2 commits intoAxial-SQL:mainfrom
Blake-goofy:feature/theme-support
Draft

Add theme support#77
Blake-goofy wants to merge 2 commits intoAxial-SQL:mainfrom
Blake-goofy:feature/theme-support

Conversation

@Blake-goofy
Copy link

@Blake-goofy Blake-goofy commented Mar 16, 2026

Currently I only added support for settings, about, and data transfer windows. Planning on doing more soon.

Before

image

After

image

@Blake-goofy Blake-goofy marked this pull request as draft March 16, 2026 23:56
@Blake-goofy Blake-goofy changed the title Add theme support for settings, about, and data transfer pages Add theme support Mar 16, 2026
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I just removed AllUsers="true" to make install easier for testing.

@alex-bochkov
Copy link
Contributor

Could you please try to reduce the code duplication where possible? I think a ResourceDictionary might help here.
The same applies to the C# code. I want to make sure it is easy to reuse and not too difficult to hook into a new form.

**ChatGPT**
1) ResourceDictionary (most common for shared UI pieces, styles, templates)
Define reusable pieces in a separate .xaml file: SharedResources.xaml

<ResourceDictionary
    xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
    xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml">

    <Style x:Key="MyButtonStyle" TargetType="Button">
        <Setter Property="Background" Value="LightBlue"/>
    </Style>

</ResourceDictionary>

Reference it in another XAML:

<Window.Resources>
    <ResourceDictionary>
        <ResourceDictionary.MergedDictionaries>
            <ResourceDictionary Source="SharedResources.xaml"/>
        </ResourceDictionary.MergedDictionaries>
    </ResourceDictionary>
</Window.Resources>

@Blake-goofy
Copy link
Author

Yes, I will keep that in mind moving forward. Thanks for the feedback.

@Blake-goofy
Copy link
Author

@alex-bochkov, I took the liberty of attempting to harmonize the tools window headings to match the GitHub sync window. I like the link to the wiki (see image in the top comment).

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