yuri-becker / react-jigsaw-puzzle

🧩 Simple plug-and-play React component for jigsaw puzzles
https://yuri-becker.github.io/react-jigsaw-puzzle/
GNU General Public License v3.0
24 stars 14 forks source link

feat: Add hint functionality to JigsawPuzzle component #30

Open Ajwebdevs opened 1 week ago

Ajwebdevs commented 1 week ago

I recently worked on a custom game project where we were looking for puzzle libraries. I found one that seemed promising, but unfortunately, it didn't have the hint feature we needed. It took me some time to extract the main code from the node_modules and rework it a bit to add the hint feature, allowing it to be passed as a prop and used for providing hints in the game. I'm looking forward to having more open discussions about this. If there's anything missing or if you have questions, please let me know. It was really fun to work on this!

yuri-becker commented 1 week ago

Hi Arjun,

Thank you so much for submitting this. I will look into this in more detail in the following days, and hopefully merge it. For now, could you re-commit your changes but without the formatting changes? Running the linter with the --fix flag should (hopefully) fix the formatting.

Ajwebdevs commented 1 week ago

Hello Yuri,

I hope you're having a great day! Thank you so much for responding. I will definitely look into that and recommit the changes. I overlooked an issue with the linter it auto-formatted the code for me. I will investigate the problem and recommit soon.

Edit:I've committed the new changes I hope this resolves the issue. If there are any further concerns, please let me know. Have a great day!

yuri-becker commented 1 week ago

Hi @Ajwebdevs,

The formatting is still changed and therefore every single line of code is changed. It's very hard for me to review the code like this, not to mention that eslint will throw errors in this state.