@IamArshadAli
Posted
Hello There! 👋
Congratulations on completing this challenge. 🎉
I've some suggestions that might interest you. 💡
-
use
overflow: hidden;
on your.card
, it will take care of the overflowing image, and the specifiedborder-radius
will be applied to the card. -
you can add a screenshot into the readme like this
![alt text](./src/images/screenshot.jpg)
or this<img src"./src/images/screenshot.jpg" />
Hope this helps you. 👍
Happy Coding 🤓
Marked as helpful
@gosiast
Posted
Hi! @IamArshadAli :) Thanks a lot for your help! Fixed now! :)
Do you know why the screenshot solutions break my "change" link? I checked, and it looks correct on my machine.
Thanks and have a nice day!
@IamArshadAli
Posted
Greetings @gosiast The challenge's screenshot provided to us are captured at a width of 1440px
for desktop and 375px
for mobile screens. So, it's important to pay special attention to these screen sizes.
You might have a display with a screen resolution of 1920x1080
or something else, which is why the design looks perfect on your machine and breaks in the screenshot, as the screenshot mechanism looks for our design at 1440px
width.
You might want to consider simulating your screen using the developer tools of your browser. Most of the browsers follow the same path to access the developer tools i.e. options > more tools > developer tools
or you can simply use the following key combination Ctrl (or Cmd) + Shift + I
.
Then set the dimensions of the screen to the desired size and you are good to go.
Hope this helps. Thank you🤓
Marked as helpful