Sudoku review #1

Open
BordedDev wants to merge 19 commits from BordedDev/sudoku:main into main
pull from: BordedDev/sudoku:main
merge into: retoor:main
Conversation 3 Commits 19 Files Changed 8 +3 -1
1 changed files with 3 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 70a27f1089 - Show all commits

4
reviews/BordedDev/notes.md
View File

@ -29,4 +29,6 @@ Borders are on all grid elements, causing a double border on the edges
I'd recommend running Biome.js or ESlint + Prettier to clean up the code
(if you're using ML you can even make these strict enough about ordering and naming conventions)
When you create an object where the key's and value's name are the same, you can use the shorthand `{ value }` instead of `{ value: value }`
When you create an object where the key's and value's name are the same, you can use the shorthand `{ value }` instead of `{ value: value }`
Border radius doesn't propagate to the children, so the cells are still square and will show if it has a background