Responsive 3-column preview challenge solution

Hello there, Richard! 👋

What is the thing that going on with the attribution on desktop view? I don't see any issue.

I notice an empty script.js file in the GitHub repository which I highly recommend removing. I might make other developers (who might be yourself in the future) wonder about it.

Also, all the icons aren't showing up because it's actually trying to find all icons on the root domain which in this case https://detomdev.github.io/. So, the full path would be https://detomdev.github.io/images/icon-sedans.svg. I recommend adding a . before the forward-slash (./) to make the start point at the current directory.

All the icons are decorative images so I suggest leaving the alt="" empty. This way, screen reader users will only listen to the page content.

That's it! I hope this information is useful! 😁

Hello there, Richard! 👋. Good job.

you can do simple improvements for this challenge:

• instead of using border-radius separately for .sedans and .luxury divs and change them again by mediaquery. You can set border-radius just for the .container and everything will work fine.

	.container {
		border-radius: 10px;
		overflow: hidden; /* to crop radius perfectly */
	}

• For active state for button elements. you can change cursor of the mouse to pointer to inform user this is clickable

	button:hover{
		cursor: pointer;
	}

• To be more HTML Semantic you can use <footer> element instead of .attribution div.

I hope this wasn't too long for you, hoping also it was useful😍.

Goodbye and have a nice day.

Keep coding🚀