jmderas / towers-of-hanoi Goto Github PK
View Code? Open in Web Editor NEWThe objective of the game is to move all the blocks from the left most column to the right most column.
Home Page: https://jmderas.github.io/Towers-Of-Hanoi/
The objective of the game is to move all the blocks from the left most column to the right most column.
Home Page: https://jmderas.github.io/Towers-Of-Hanoi/
The div in the HTML is already a <footer>
, so you can simply use footer
instead of .foot
as the CSS selector. One of the advantages of the semantic HTML elements
the attribute value "hidden" for the class
needs to be wrapped in quotes. Browsers are helpful in that they can fill in missing characters like this, but if there was more than one class that fallback most likely would not work correctly so it is important to use proper syntax even if it "works" without it
For the rings, it could be useful to have a single .ring
class and corresponding CSS styles for common properties like the background-image
and height/width
etc
Indenting the code is awesome, but in this section and at least one other place the text is indented a few levels too far. Usually when writing the code line by line the editor (VSCode or otherwise) is good about inserting the correct indentation but if you're moving stuff around it can get wonky
You do a great job of listing the references in the Javascript, but the animation here looks like it's from somewhere. It's very important to always cite external sources, and I'm not pointing fingers I just think that if you did cite the animation I must have missed it.
One of the project requirements is to commit often throughout the project. There are not many commits for this project, and what is here gave me pause.
Commit f641413 adds a lot of new code, particularly in the JS. I do see that in Commit 7f226f8 you added your sources, which is absolutely what we need to see. I'm not pointing fingers, and I know you are a solid student from working with you and have no doubts about your personal integrity. But the frequent commit requirement is there for several reasons, and it is helpful to see many small commits over the course of large projects to give us insight into your process.
In the future I ask and advise you to make commits as you make progress so that we have a better idea what were stumbling blocks and how you solved them in your submission.
It is likely you could use a single Event Listener on the parent container to account for all three towers. Three individual event listeners works just fine, but looking at the code I think of DRY (Don't Repeat Yourself) and feel that with a little time and attention you could write a more robust single function callback for the event handlers here.
A declarative, efficient, and flexible JavaScript library for building user interfaces.
๐ Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
An Open Source Machine Learning Framework for Everyone
The Web framework for perfectionists with deadlines.
A PHP framework for web artisans
Bring data to life with SVG, Canvas and HTML. ๐๐๐
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
Some thing interesting about web. New door for the world.
A server is a program made to process requests and deliver data to clients.
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
Some thing interesting about visualization, use data art
Some thing interesting about game, make everyone happy.
We are working to build community through open source technology. NB: members must have two-factor auth.
Open source projects and samples from Microsoft.
Google โค๏ธ Open Source for everyone.
Alibaba Open Source for everyone
Data-Driven Documents codes.
China tencent open source team.