Update icons to use UnifiedIcon

[erri120]

Resolves #979.

[Al12rs]

I see that there are still a lot of icons:Icon around, specifically inside buttons.
I think we need those ported over and the button styles updated to use UnifiedIcon instead of icons:Icon

Apart from that I think there might be broken due to the FontSize vs Size thing.
Game Widget had an Icon on the Add Game button that I think was broken even before this PR and needs updating.

[erri120]

I see that there are still a lot of icons:Icon around, specifically inside buttons. I think we need those ported over and the button styles updated to use UnifiedIcon instead of icons:Icon

I see them now. I was updating the files by searching for the Projektanker namespace, but apparently some files are using xmlns:icons="https://github.com/projektanker/icons.avalonia" instead. I'll fix those.

Apart from that I think there might be broken due to the FontSize vs Size thing. Game Widget had an Icon on the Add Game button that I think was broken even before this PR and needs updating.

A lot of icons didn't have sizes set before, and I didn't want to touch the styling. I changed FontSize to Size and I can fix some sizes.

[Al12rs]

A lot of icons didn't have sizes set before, and I didn't want to touch the styling. I changed FontSize to Size and I can fix some sizes.

Hmm, I'm worried we might have issues if we put something that isn't a projektanker icon in the ones that don't have a size and suddenly break the UI with too big or to small icons.
I'm going on the assumption that projektanker icons have a default size while our UnifiedIcon doesn't depending on the underlying type.

[codecov-commenter]

Codecov Report

Attention: Patch coverage is 66.66667% with 2 lines in your changes are missing coverage. Please review.

Project coverage is 58.36%. Comparing base (596d997) to head (bc8e1e5).

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #991      +/-   ##
==========================================
+ Coverage   58.18%   58.36%   +0.18%     
==========================================
  Files         637      637              
  Lines       20564    20562       -2     
  Branches     1572     1571       -1     
==========================================
+ Hits        11966    12002      +36     
+ Misses       8240     8201      -39     
- Partials      358      359       +1     

Flag	Coverage Δ
Linux	57.56% <66.66%> (+0.15%)	⬆️
Windows	57.56% <66.66%> (+0.18%)	⬆️
clean_environment_tests	58.35% <66.66%> (+0.18%)	⬆️
macOS	56.99% <66.66%> (+0.12%)	⬆️
network_tests	?
networking_tests	?

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

Files	Coverage Δ
...exusFluentDark/Styles/Controls/Border/Toolbar.xaml	97.50% <100.00%> (ø)
...sFluentDark/Styles/Controls/Button/PillStyles.xaml	98.91% <100.00%> (-0.03%)	⬇️
...s.App.UI/Controls/GameWidget/GameWidgetStyles.xaml	0.00% <0.00%> (ø)

... and 6 files with indirect coverage changes

[erri120]

A lot of icons didn't have sizes set before, and I didn't want to touch the styling. I changed FontSize to Size and I can fix some sizes.

Hmm, I'm worried we might have issues if we put something that isn't a projektanker icon in the ones that don't have a size and suddenly break the UI with too big or to small icons. I'm going on the assumption that projektanker icons have a default size while our UnifiedIcon doesn't depending on the underlying type.

We shouldn't rely on implicit default sizes of Projektanker icons anyway.