mstru55• 150
@mstru55
Submitted
dadkinstech
@dadkinstech
All comments
- dadkinstech• 40
@dadkinstech
Posted
Social card is a bit too tall, leaving some empty space. Other than that, looks great!
0 - Victor Sobrinho• 110
@victorsobrinho93
Submitted
dadkinstech• 40@dadkinstech
Posted
Looks great in a variety of resolutions, containing all the contents, but there's a bit of a jump as you pull the width past 960. Other than that, it looks great!
0 - Gabriel Ndibe• 10
@frickygaby
Submitted
dadkinstech• 40@dadkinstech
Posted
On the right track. It took me some time to fiddle with fitting the text in the bottom section. I ended up using something along the lines of min-content if that helps.
0 What are you most proud of, and what would you do differently next time?
This was my first (intentional) responsive website ever! I'm proud of how i was able to make the design as close as i could.
What challenges did you encounter, and how did you overcome them?
I had a problem where, once the screen was wide enough, the image would change to the desktop version and take up the entire screen. The weird thing is this didn't happen on Firefox (it worked as intended). I ended up fixing it by updating the image's height to auto. I believe what happened was that I set the height to auto once to test it on Firefox and then removed it to see what would happen. It still worked, so I didn't use it again. My guess is that Firefox saved the image's height in cache and was able to use that
What specific areas of your project would you like help with?
Could you please review my CSS code and provide feedback on the following aspects:
Code structure and organization Cross-browser compatibility Responsive design Flexbox usage Performance Styling consistency Best practices
I'm particularly interested in any suggestions for improving the layout and ensuring it works consistently across different browsers and devices."
dadkinstech• 40@dadkinstech
Posted
Looks great! I really like the background color you went with.
1
