HTML5, CSS3

Hi Zaib,

The most valuable quality of a developper is his capacity to solve problem by himself. At junior level, most of the solution can be find by googling the issues. Per example, for your hover problem you could make a search on Google with the keywords : "image hover css". The first results answer what you were looking for. This is the best way to improve.

Hope it helps,

Keep up !

Hello @iamzakofficial,

I have some suggestions:

• Anything with a hover style in the design means it's interactive . You need to add an interactive element around theimage , Equilibrium #3429 and Jules Wyvern(in this challenge is an anchor tag <a>)

• The images alt ="time" alt="eth" don't need filled alt. They are decorative images, so alt attribute should be left intentionally blank. You can optionally addaria-hiddenor role presentation to ensure the images are always ignored by screen readers AND image-equilibrium.jpg" and avatar's one shouldn't be empty read up about how and when to write alt text

• <span class="nft-title highlight">Equilibrium #3429<br></span> no need for a span you can use a heading.

• The eye image doesn't really need to be in the html, you could do it with css.

• The last part of the card you can use some semantic tags like:

<figure class="">
    <img class="avatar-img" src="images/image-avatar.png" alt="">
<figcapton class=""> Creation of <a href="#">Jules Wyvern</a></figcaption>
</figure>

• You can simply use unordered list <ul>to wrapclass="info"and in each list item would have <img > OR <Svg> and <p>.

• No need for this <hr />as you can use border-top to the <figure class="">.

• You should use em and rem units .Both em and rem are flexible, scalable units which are translated by the browser into pixel values, depending on the font size settings in your design. Using px won't allow the user to control the font size based on their needs.

• Try flexbox properties and a min-height 100vh on it to center the card.

body{

    font-family: 'outfit', 'sans-serif';
    font-size: 18px;/* Never use font-size in pixels*/
    /* max-width: 1440px; */
    background-color: hsl(217, 54%, 11%);
    display: flex;
    align-items: center;
    justify-content: center;
    flex-direction: column;
    min-height: 100vh;
    width: 100%;
}

Hopefully this feedback helps.

Hi, @iamzakofficial ~

Congratulate on your solution to the challenge on FM platform. I have studied your work carefully and learned a lot from it.

Here are some of the tips I like to provide.

1. I hope you to add transition effect for heading and images part of the card element.
2. Adding margin: auto property to <main> element will center your card.
3. You can add max-width: 100% for image tags on this project.
4. On smaller devices, we need responsiveness. That's important part here.

https://www.frontendmentor.io/solutions/my-first-solution-on-chanllenge-V-4IzAivH

Here is my solution to this challenge, and if it can help you even a bit, it would be happy to me.

Cheers ~