@abdelrahman-source
Latest comments
- @msshaikh1717
Assalamu Alaikum brother,
Remove these from body tag: min-width: 1440px; min-height: 100vh;
As far as I know, you should avoid using vh and px. Use rem and %. I have a 1920x1080p monitor so your min-width:1440px have a horizontal slider in the page which is the worst thing(horiz slider) to have. You dont really need to specify width and height much in this challenge.
And brother, you dont need tp specify 1st, 2nd, 3rd etc pictures and all. Just do any one and add all into it like:
.daniel, nkira, jonathan {}
For images, this will work for all images: img {}
Marked as helpful - @zune-moe@msshaikh1717
plz look here https://github.com/msshaikh1717/four-card-feature-section.git Only this is enough: No need to specify gtr and other stuff:
@media screen and (min-width: 768px) { .cards { display: grid; grid-template-columns: repeat(3, 1fr); align-items: center; }
#calc { grid-column: 3 / span 1; }
#super, #calc { grid-row: 1 / span 2; } }
Marked as helpful - @zune-moe@msshaikh1717
When viewing live site, Your Supervisor card is on top and Calculator card is on bottom. Your should fix that
- @codeMcTech@msshaikh1717
Finally someone whose code I can read and understand. I believe most people here are not using their brain and taking AI to generate HTML $ CSS. Good work bro, I see that your code is still incomplete(missed 'price' part). Hopefull we will be able to help each other out.
.product-img { content: url("images/image-product-mobile.jpg"); } This was helpful to me and better approach IMO. I took display:none route :)
- @eekaa29@msshaikh1717
good
- @Ig0rSPWhat specific areas of your project would you like help with?
It's appropriate to use 'nav' instead of 'div' in this context?
@msshaikh1717Good
