brollin
Refactor
I got carried away. May I present every maintainers favourite PR: a sweeping factor with no test coverage!
I have manually tested all of the grid types and the boxes and points functions, as well as rulers. I don't think I'm expecting you to read through and review this entirely. If you are happy to just test the functionality, I think this would be a net improvement on what was otherwise a very messy codebase; it will make any future maintenance significantly easier.
I tried to keep behavioral changes to a minimum, but there were one or two pieces of code that I explicitly removed that might possibly have edge cases missed. I have also removed the code that made the active superblocks slightly different colours depending on which of the blocks it was, which seemed pointless to me and seemed like it was probably left over from legacy. Let me know if you think that behavior was actually desirable and I will revert it.
