@James-alderson
Posted
Hi Cristina Kelly, I saw your design, it was very well designed, but there are a few small issues that you can fix to help make your design better.
as follows:
- Create a folder called 'assets' and put all the files related to the challenge inside it.
- Define your web fonts inside css, by doing this you have helped the HTML code to be seen better, and in addition, the challenge you have designed will load faster because the fonts will be loaded after the design is fully loadedุ and until then it uses the system font for display.
- It is better to use the 'a' tag for texts that can be hovered.
- Use the 'table of content' above your css code, this will help your css codes to be better seen and clean.
- Challenge design for mobile resolution.
- Adding design screenshots to challenge files.
- Do not ignore the 'README' file, because it displays information about your design.
- ...
By the way, I have created a fork in your repository, you can add all the changes to your project this way.
These are some of the tips that help to improve your design, you can see all the changes in the commits section.
Don't forget to practice, because the more you practice, the better and faster you will design.
Happy coding :)
Marked as helpful
@cristinakellyt
Posted
Hi James! Thank you for your time reviewing my code and for all your suggestions! I really appreciate them, and for sure will keep them in mind. As you said, these are small issues but it does a lot of difference. I am going to see it right now on Github!