feat: Theming - Inline svgs

[bmingles]

No description provided.

[dsmmcken]

would I then have to specify an INLINE_THEME.TS file to replace the icons if I, as a third party theme write was trying to create one?

I imagined it would have just been like variable as part of theme:

--icon-whatever: url("svg path blah blah blah fill="var(--some-color)"...")

and then before you insert the variables/on theme change you resolve it to the actual color

[bmingles]

would I then have to specify an INLINE_THEME.TS file to replace the icons if I, as a third party theme write was trying to create one?

I imagined it would have just been like variable as part of theme:

--icon-whatever: url("svg path blah blah blah fill="var(--some-color)"...")

and then before you insert the variables/on theme change you resolve it to the actual color

@dsmmcken You can just define the svg css var in the theme .css files. Since it gets loaded after the inlinesvgs, it will take precedence

[codecov]

Codecov Report

Attention: 34 lines in your changes are missing coverage. Please review.

Comparison is base (c0cc966) 46.62% compared to head (13663d9) 46.66%.

❗ Current head 13663d9 differs from pull request most recent head 5e7846b. Consider uploading reports for the commit 5e7846b to get more accurate results

Files	Patch %	Lines
packages/code-studio/src/styleguide/colorUtils.ts	27.90%	31 Missing ⚠️
...ackages/code-studio/src/styleguide/ThemeColors.tsx	0.00%	3 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1651      +/-   ##
==========================================
+ Coverage   46.62%   46.66%   +0.04%     
==========================================
  Files         597      599       +2     
  Lines       36620    36660      +40     
  Branches     9172     9184      +12     
==========================================
+ Hits        17073    17107      +34     
- Misses      19495    19501       +6     
  Partials       52       52              

Flag	Coverage Δ
unit	46.66% <59.03%> (+0.04%)	⬆️

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

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[mofojed]

Just a little bit of cleanup. Looks good overall.