Skip to content

fix: theme-aware issue on background#12

Merged
CalvinAllen merged 1 commit intoCodingWithCalvin:mainfrom
timheuer:main
Feb 14, 2026
Merged

fix: theme-aware issue on background#12
CalvinAllen merged 1 commit intoCodingWithCalvin:mainfrom
timheuer:main

Conversation

@timheuer
Copy link
Contributor

  • make toolbar window background theme-aware
  • change launch settings do use devenvvar in case someone doesn't have Community installed

Description

Fixes #11 by adding the vs brush for toolwindow background

Also, I wasn't able to immediately clone and debug because of hard-coded path in launchsettings. Moved to use csproj and envvar so that anyone could use it regardless of Community, Insiders, etc. -- optional fix, but this helped me.

Type of Change

  • feat - New feature
  • fix - Bug fix
  • docs - Documentation only
  • refactor - Code change that neither fixes a bug nor adds a feature
  • test - Adding or updating tests
  • chore - Maintenance tasks
  • ci - CI/CD changes

Related Issues

Fixes #11

Checklist

  • My code follows the project's code style
  • I have tested my changes locally
  • I have updated documentation if needed
  • My commits follow the conventional commit format
  • This PR has a descriptive title using conventional commit format

Screenshots (if applicable)

image image image

Additional Notes

- make toolbar window background theme-aware
- change launch settings do use devenvvar in case someone doesn't have Community installed
Copy link
Contributor

@CalvinAllen CalvinAllen left a comment

Choose a reason for hiding this comment

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

Thanks!!

🚀

@CalvinAllen CalvinAllen changed the title Fix theme-aware issue on background bug(ui): fix theme-aware issue on background Feb 14, 2026
@CalvinAllen CalvinAllen changed the title bug(ui): fix theme-aware issue on background fix: theme-aware issue on background Feb 14, 2026
@CalvinAllen CalvinAllen merged commit 67fe4f8 into CodingWithCalvin:main Feb 14, 2026
2 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

Not honoring theme

2 participants