Chris Hennes

Results 699 comments of Chris Hennes

Obviously at 2000+ lines of code, there are more tests that could be written, but that is true of our entire codebase. I'd be OK with merging this to main...

I'm not sure about international laws on copyright, but that's not how it works in the US. Nevertheless, back when the logo was created I did ask the author (@sytabaresa)...

We are going to drop the `[readability-identifier-length` linter check, so we are ignoring that one here.

Pinging myself to do this review... @chennes

@kadet1090 is going to take a look at doing the final polish here.

I think I have resolved the conflict correctly, but I need someone else to check it to ensure I didn't choose the wrong code path.

The problem is that this PR does not take into account the changes made in #16438 -- so it needs more than a simple merge or rebase, it needs to...

`QSettings` is mostly used to store the stuff that Qt wants to store there (toolbar positions, window positions, etc.) while it would be delightful to rip it out entirely, that...