Daniel Cazzulino

Results 360 comments of Daniel Cazzulino

![3412 passed](https://img.shields.io/badge/✅-win‐x64%20in%2030s-blue) ![3412 passed](https://img.shields.io/badge/passed-3412-brightgreen) ![8 skipped](https://img.shields.io/badge/skipped-8-silver) :test_tube: Details on Microsoft Windows 10.0.20348 from [dotnet-retest](https://github.com/devlooped/dotnet-retest) v0.7.1 on .NET 8.0.16 with [:purple_heart:](https://github.com/sponsors/devlooped) by @devlooped

Hi @ndwpatrick. Help is very much appreciated!

I totally need this. I added my own "poor-man's" toggler but this looks much better.

A minor improvement to this: currently, when the theme is set from local storage, the button icon/text isn't updated to match. This means as you navigate around, you end up...

I don't think the reset is necessary, TBH. In my customization, I removed that from this PR, but it's a matter of taste, I think.

I think `SetReturnsDefault` satisfies specific needs already, without introducing the potential for severe breaking changes if the current behavior were modified. I don't think that will happen.

Why do you consider a warning to be agressive? Informational messages are typically not even read, are they? In addition, the warning *never* causes a build error, so it's just...

Run `dotnet build` from a regular terminal, or CI, and you'll see there are no warnings in those cases. I'm explicitly accounting for "no warnings" policies so it should never...