@varunsharmablog
Posted
Hey Yahir, Congratulations on your solution.
I looked at your code and I believe the way you did the border radius absolutely matches the solution. Infact, I have adopted it into my own solution. I was defining the border radius with percentages, but when I defined it with pixels, it just looked so much better. I wish I had more free screenshots for this month so that I could compare the design and my solution after changing the border radius.
As far as the dimensions go, If you can get the containers dimensions to be 320x497 px and the QR code images dimensions to be 288x288 px, it would be identical to the dimensions of the solution. Your current dimensions are 311x490.89 px for the container and 279x279 px for the QR code image which is why it looks a bit different.
Also, wrap the container in the <main> tag for accessibility.
Other than that, your code seems perfectly fine. Good job!
Marked as helpful
@Le-Yzzz
Posted
@varunsharmablog thank you!!!! i always struggle with dimensions 😅 how do you get them so perfect?
@varunsharmablog
Posted
@eurus1108 commented on my solution for the QR Code and I looked at his solution. His solution was identical to the design itself and so I inspected the dimensions on his live page. From there I tried to emulate those dimensions into my own code. Just like how I adopted your border radius values.
But you do raise a good question. I wonder how @eurus1108 figured out the dimensions.
@oshudev
Posted
@varunsharmablog I figure out the dimension by recreating the design itself.
@varunsharmablog
Posted
@eurus1108 I see! so you are guessing the values as well?
@oshudev
Posted
@varunsharmablog yeah