Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Colors and Fonts resources #736

Merged
merged 3 commits into from
Oct 25, 2023

Conversation

erri120
Copy link
Member

@erri120 erri120 commented Oct 25, 2023

Part of #735.

@erri120 erri120 added the Design UI/UX This is related to the UI. label Oct 25, 2023
@erri120 erri120 requested a review from a team October 25, 2023 14:48
@erri120 erri120 self-assigned this Oct 25, 2023
@erri120 erri120 mentioned this pull request Oct 25, 2023
10 tasks
Comment on lines +15 to +17
<ResourceInclude x:Key="Colors" Source="avares://NexusMods.App.UI/Palatte/Colors.axaml"/>
<ResourceInclude x:Key="Fonts" Source="avares://NexusMods.App.UI/Palatte/Fonts.axaml"/>

Copy link
Contributor

@Al12rs Al12rs Oct 25, 2023

Choose a reason for hiding this comment

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

The idea is that we have everything in a single file (Nexus.xaml) that then references the rest.
We want to be able to swap out one file and get a completely different styling for the app.

Is that possible?

Copy link
Member Author

Choose a reason for hiding this comment

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

You can just load a different resource file later and it will overwrite the previous values.

@codecov
Copy link

codecov bot commented Oct 25, 2023

Codecov Report

Merging #736 (0d61338) into main (de31432) will increase coverage by 0.71%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #736      +/-   ##
==========================================
+ Coverage   63.26%   63.98%   +0.71%     
==========================================
  Files         581      581              
  Lines       17796    17796              
  Branches     1247     1247              
==========================================
+ Hits        11259    11386     +127     
+ Misses       6205     6088     -117     
+ Partials      332      322      -10     
Flag Coverage Δ
Linux 63.17% <ø> (-0.10%) ⬇️
Windows 63.16% <ø> (?)
clean_environment_tests 63.95% <ø> (+0.69%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

see 30 files with indirect coverage changes

@halgari halgari merged commit b02b07c into Nexus-Mods:main Oct 25, 2023
6 checks passed
@erri120 erri120 deleted the issue-735-style-to-resources branch October 26, 2023 08:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Design UI/UX This is related to the UI.
3 participants