Tiny Component With Vanilla JS

Solution retrospective
Hey, everybody!
This is my first challenge with JS, and it was fun! 😃
The JS was easy, but the CSS took me a little longer to code than I initially thought. 😅
Feedback on both the design and code is welcomed and appreciated! 😊
And as always, happy coding! 🎉
Please log in to post a comment
Log in with GitHubCommunity feedback
- @correlucas
This comment was deleted almost 3 years ago
- @adityas24
Hey in your design i saw that the size the of the card remains the same, i mean that it looks like that you had set some fixed width and height. And also it shifts to mobile design quite early at around 1000px. I want to ask that is it OK to do it like that. I am a newbie here. I am trying to make it responsive for every possible point and shift to mobile design atleast after 500px . But its a mess for me that way. Please help me out.
- @Darknessflowers
Amazing job at getting this pixel perfect to the design! It looks so slick.
- @LaBoiteDeCarton
Like tarasis said, you nailed it. I think the only thing missing, is the btn design when you click on it. For me the background color should change and stay fixed even when unhovered, and the same for svg color. I learned a lot from your CSS. (and felt stupid when saw your js, so clean)
- P@tarasis
👋 ApplePieGiraffe
You absolutely nailed the design. Especially that pesky image!
Visually the only quibble is the mobile design on a iPad screen using Compact Width (can’t remember the dimensions ottomh). The share button is right up on the Pinterest icon.
Need to learn from your CSS. Curious about the setting
font-size
in html to 50%, why that than an explicit figure? - @CSumm
Looks great!
Join our Discord community
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord