Hello, this is my first challenge on here, and truthfully it took me a bit to complete!
In terms of feedback, I was wondering if anyone would let me know if there are any "no-no"s I committed (was my code written to the standard it should have been) with my HTML (was it not semantic, etc.) - I tried my best to use <divs> in the right spot, I do believe I used both a class and an ID when I believe I could have just used one of them.
Yeah! Apologies for it being super ugly and super messy. :( and thanks in advance to anyone who answers my question, I appreciate it.
Set the font size to 15px (no big deal, but it's always best to follow design recommendation).
Instead of using div around main class, you could use main for accessibility purposes.
The last thing and the most important is that your solution is not responsive. The QR-code is not visible on mobile. Kindly update your code to match the mobile design. 🙂
Resource.Responsive design
I hope this is helpful. Feel welcome to let me know if you have any questions.
#Color matching.-
The main container color on your
solution is #252e37 while the deign is #1f2630
To match design colors if not specified, I have been using the Firefox Eyedropper function available on the Firefox browser console.
