CairoGarb• 810
@CairoGarb
Submitted
+1 project done! Any tips or tricks to improve the code will be appreciated, thanks!
P
Zephyr_Grey
@Mercury225
All comments
- PZephyr_Grey• 220
@Mercury225
Posted
Hi there! The HTML code is clear and uses semantic marking. I would look more into accessibility coding for HTML (aria roles), as well as adding name attributes for input tags. I would also try to reduce the CSS, there are 1800 lines of code for this website. Keeping code simple and short will help people navigate through your code. Otherwise great job!
Hope this helps!
Marked as helpful
0 - Aasim• 50
@080Aasim
Submitted
Please provide feedback.
PZephyr_Grey• 220@Mercury225
Posted
Looks all good to me! The only thing is the heading a bit bigger than the original, i would lower the font-size by a little.
<body><h1>Jessica Randall</h1><h4>London,United Kingdom</h4></body>h1 22px h4 ( should be h2) 18px
0 - Dêvid Teófilo• 240
@devid8642
Submitted
About the colors used in this project. I applied the color "light gray" to the text "Scan the QR code to visit Frontend Mentor and take your coding skills to the next level" and it was very faded in relation to the background color of the card. So I had to use a darker shade of gray. What do you think about this, was this shade of gray the right one and was I doing something wrong about applying it?
PZephyr_Grey• 220@Mercury225
Posted
Hi there, Great job! I'm having a look at your code and there are some things that stick out to me:
In index.html you write your CSS in your html
-It's common practice to put this code in another CSS file (call it maybe styles.css) and to put a HTML link tag to the CSS file.
- margin: 50px auto 0; is better
-Using px is fine, but for responsive design purposes, it's usually a better habit to use the unit rem, or em, to make it easier for resizing for bigger projects
And about the shades of grey, the darker background grey looks a bit sinister, i would stick with a lighter shade
Hope this helps!
Marked as helpful
2
