@Dsimpson91
Posted
Hi Kishore,
This is looks great don't be so hard on yourself when it comes to css it sucks lol. No seriously there's multiple ways to do things using css. For an example i used flexbox your css looks good to me and it's responsive. I'll give you a tip to help keep it cleaner instead of doing this
.qr-section-info { margin-top: 24px; padding: 0px 16px 0px 16px; }
.qr-section-info h2 { color: rgb(31, 49, 79); margin: 0 16px 16px 16px; font-size: 22px; font-weight: 700; line-height: 1.2; }
.qr-section-info p { color: rgb(104, 119, 141); font-size: 15px; line-height: 1.4; font-weight: 400; margin: 0 16px 16px; }
add everything under your selector like this
.qr-section-info { margin-top: 24px; padding: 0px 16px 0px 16px; p { color: rgb(104, 119, 141); font-size: 15px; line-height: 1.4; font-weight: 400; margin: 0 16px 16px; } h2 { color: rgb(31, 49, 79); margin: 0 16px 16px 16px; font-size: 22px; font-weight: 700; line-height: 1.2; } }
this will make things easier to maintain once you start working on larger projects my last comment is just practice flexbox, css grid, and sass it will make things easier. Great job !!!!!
