search

AmineHLub / Microverse-Portfolio

This is Microverse's first module project. It's a figma replication attempt. It's a template portfolio
2 stars 0 forks source link

peer-to-peer code review #6

Open see-why opened 3 years ago

see-why commented 3 years ago

use of buttons tags for the language literals on index.html, we think it would be better to use ul and li tags, change the height of the background to pixel in place of percentage

Marcraphael12 commented 3 years ago

use of buttons tags for the language literals on index.html, we think it would be better to use ul and li tags

good job so far guy, but it would be a good thing for your code to use div tag instead of repeating section tags on your cards project. sections tags are supposed to be the different part of your html and div are supposed to devide those parts.