@erickd3v
Submitted
@webdevhill
@erickd3v
Submitted
@webdevhill
Posted
that's cool how you rendered that with forEach
@hannahvw
Submitted
@webdevhill
Posted
For this simple card I hard coded the width and height on desktop, 1080 x 446px. I like your use of a psuedo element for the overlay. Clean Code!
@MTNguyen12
Submitted
@webdevhill
Posted
That is spot on. Clean code. Nice work
@webdevhill
Submitted
@webdevhill
Posted
I still am having trouble with my screenshots. Things look good in comparison with the previews but I am having trouble positioning the <main> component on the viewport.
Maybe I should use Firefox for my screenshots?
@WojciechPodlesny
Submitted
@webdevhill
Posted
consider wrapping your container in a <main> element and providing a viewport height for it, the 100vh on your HTML has no effect
that should get is much closer for comparison
there is a small typo
hope that helps, Happy coding!
@Gudymedia
Submitted
@webdevhill
Posted
GitHub is not enforcing HTTPS on your page
gudymedia.github.io throws an unsecured warning, here are the GitHub docs for HTTPS
https://docs.github.com/en/pages/getting-started-with-github-pages/securing-your-github-pages-site-with-https
@anetakak
Submitted
@webdevhill
Posted
extremely close to an exact solution
The responsiveness works very well.
nice work!
@phislipe
Submitted
🤖 Hi guys! Here's my solution to the "QR code component" challenge!
Came back to this project after a while to improve the code, made some major changes in the CSS, and implemented display: grid
instead of flex
.
♻️ Changes:
As a newbie, I'll be happy to hear any feedback and advice!
@webdevhill
Posted
The proportions and size are spot on. Is it responsive below 375px? Not really, but it is a clean solution to the challenge.
Well done!