@En-Jen
Posted
Hey Patel, nice solution. Looks pretty close to the design :) I just have a few suggestions.
-
Your images look a bit squashed (like the right and left sides have been squeezed together). You can avoid this by giving the width a value and setting
height: auto;
-
The features are not centered at all screen widths.
-
In future projects, I would try to avoid using id names for CSS selectors and instead use class names. You can read more about the rationale for that here: https://paulcpederson.com/articles/css-for-people-who-hate-css/
-
On screen widths below 1128px, the features section and the productive section could use more whitespace between them.
Keep it up! -Jen
@En-Jen i would like to contact you Can i get your contact details
@En-Jen
Posted
@moKabir feel free to drop me a message on the Frontend Mentor Slack channel. I think you should be able to find me if you search for “Jen Speak”