Submitted over 2 years ago

article preview component master

@yazanMhussein

A solution to the Article preview component challenge

Solution retrospective

I have a question is there a better way keep the button and social media together while the page size change.
any give feed back be great thanks.

Code

Please log in to post a comment

Community feedback

Adriano•42,870
@AdrianoEscarabote
Posted over 2 years ago
Hi yazan hussein, how are you?

I really liked the result of your project, but I have some tips that I think you will enjoy:

As you know this page is just a component, so it doesn't even need an h1! because we don't know how important the other components of the site will be! but it's always good to prevent accessibility errors so I think it would be good for you to add an h1 in this component, besides being a good practice for when you are developing larger sites, don't worry forget about h1.

to stop the content from stretching at higher resolutions, we can use a max-width and a margin: 0 auto; to center!
```
.mainBody {
    max-width: 900px;
    margin: 0 auto;
}
```
The rest is great!

I hope it helps... 👍
Marked as helpful

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord