A double filter, it's inefficient (since both will loop through the array) but for the size it doesn't really matter
@BordedDev nice review so far! I will check your new files tomorrow. I'm now fighting with speech recognition and command execution. It's a cool project and it was open source for a while but…
Dang it I missed it, it's meant to be SUDOKU_GRID_SIZE. You did with the new Puzzle(9)
Still better than my junior. I was very surprised, learning about the CSS upgrade this year
It's here: https://developer.mozilla.org/en-US/docs/Web/API/Web_components/Using_custom_elements
In the class constructor, you can set up initial state and default values, register event…
From what I've read, it's not recommended to create elements in the constructor and instead should be done in the connectedCallback
You can combine styles as well, I find it to be a bit easier to work with on average since you can toggle and combine them when it's needed much easier.
Unfortunately, this doesn't work with dynamic inside out sizing/nested aspect-ration on firefox. Luckily, you can use em since the size is based on font size
The Row/Col classes are confusing in the current situation, since Row isn't used for more than a bag holder/event forwarding
A pretty standard, pretty robust way of handling events. I'd lean to the built-in EventTarget until the need for something better is needed.